MCRecoPart uses non-portable coordinate system
sim::MCRecoPart uses the TPC volume to define whether a trajectory point is within or without the TPC.
It hard-codes assumptions on the coordinate system and it's not compatible with multi-TPC environment.
It also does not exactly match the TPC coordinates of MicroBooNE geometry, which are used by Geant4 and as a consequence by the
simb::MCParticle with its trajectory points.
The code is at: larsim:source:larsim/MCSTReco/MCRecoPart.cxx#L26 .
#2 Updated by Gianluca Petrillo about 3 years ago
The author declined to make the needed changes.
We have two options:
- ship this code back to
uboonecodeuntil the usefulness of this code is reclaimed by other experiments
- fix the limitations, recover or write the documentation, and test it on MicroBooNE and on other detectors to validate the changes (first guess is 20 hours work)