WireCell / wire-cell-toolkit

Toolkit for Liquid Argon TPC Simulation and Reconstruction
https://wirecell.github.io/
Other
7 stars 22 forks source link

anode->face() needs ident, cleanups #265

Closed HaiwangYu closed 1 year ago

HaiwangYu commented 1 year ago

Avinay found a bug in ROI_refinement that anode->face(faceid) needs face->ident, but current faceid is face->which. Brief search shows similar API calls are only found in GridTiling, so with this fix and cares taken in GridTiling, we should be fine. Some sanity checks running TrackDepo DNN-ROI:

2023-11-29 DNN-ROI4 anode2face bug.pdf