Bug #19412
EMShowerAlg requires Backtracker service for no reason
0%
Description
Moved from ServiceNow ticket RITM0661275 by Dominic Barker:
I have recently implemented the use of EMShower_module.cc
(found in the ShowerFinder
directory in larreco
) in SBND. This uses EMShowerAlg
(found in the RecoAlg
directory in larreco
) to reconstruct Showers. Backtracker services is used in EMShowerAlg
in a private function called FindParticleID
, this is then used in a private function called FindTrueParticle
. FindTrueParticle
is no longer used in the rest of the code and so has no affect on the rest of the algorithm. Is it possible to comment out both these functions and remove the backtracker service from the algorithm, as they are no longer used? EMShower
uses only reconstructed data and should not require Monte Carlo information. Commenting out the code would allow SBND to use the module in our reconstruction chain, which is found in sbndcode
standard_reco_sbnd_basic.fcl
, without having to implement the backtracker service. I feel that the reconstruction should not depend on Monte Carlo information and this is correct thing to do.
Related issues
History
#1 Updated by Lynn Garren almost 3 years ago
- Status changed from New to Assigned
- Assignee set to Michael Wallbank
Mike, would you please take a look at this?
#2 Updated by Gianluca Petrillo almost 3 years ago
- Blocks Necessary Maintenance #19431: Remove BackTrackerService from reconstruction configuration files when it's not needed any more added