Project

General

Profile

Support #10725

Investigate merging DBSCAN algorithms

Added by Gianluca Petrillo about 5 years ago. Updated almost 5 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Gianluca Petrillo
Category:
Reconstruction
Target version:
-
Start date:
10/30/2015
Due date:
% Done:

0%

Estimated time:
8.00 h
Spent time:
0.50 h
Experiment:
-
Co-Assignees:
Duration:

Description

There are two versions of DBSCAN algorithms in LArSoft:
  1. "historical" larreco/RecoAlg/DBScanAlg.h written by Kinga (possibly unmaintained)
  2. newer larreco/RecoAlg/Cluster3DAlgs/DBScanAlg.h written by Tracy Usher

Investigation should proceed to learn whether and how these two may be merged without loss of computing and physics performance.

History

#1 Updated by Lynn Garren about 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Gianluca Petrillo

We think that larreco/RecoAlg/DBScanAlg.h should be deprecated and removed, but this needs to be confirmed.

#2 Updated by Gianluca Petrillo almost 5 years ago

  • Status changed from Assigned to Rejected

Tracy has explained that the DBSCAN in Cluster3D is, well, 3D, while the one in RecoAlg is 2D.
Possibly less-than-ideal choice of names, but they are independent and both useful.
The effort to merge them would be of dubious utility.
So, rejecting the request.

Also available in: Atom PDF