Support #10725
Investigate merging DBSCAN algorithms
Experiment:
-
Co-Assignees:
Description
There are two versions of DBSCAN algorithms in LArSoft:
- "historical"
larreco/RecoAlg/DBScanAlg.h
written by Kinga (possibly unmaintained) - 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.