Bug in Fiducial Volume Definition in Larsim
In the MCReoPart class in larsim, the fiducial volume is defined incorrectly for some experiments.
The fiducial volume is assumed to have: x_min = 0, z_min = 0, and y_min = - y_max. This is not true for (e.g.) SBND and ICARUS, and leads to MCTrack points being thrown out for monte carlo in those detectors.
Please let me know if this issue should be posted elsewhere -- I could not find an issue tracker in the larsim redmine.
#1 Updated by Gianluca Petrillo about 2 years ago
As a reminder for the people involved, the
MCReco algorithms are known1 to have been written in detector-specific way (MicroBooNE's, I believe).
It was suggested some time ago that, given the non-maintained state of those, they be moved to
1 Meaning, the issue has been already discussed in the past: this does not mean that LArSoft users (nor Greg) are expected to know.