EMShowerAlg requires Backtracker service for no reason
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 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
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.