New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DNN ROI multi-plane matching wireplaneId != faceid #251

Open

weihythu opened this issue Oct 7, 2023 · 0 comments

Member

weihythu commented Oct 7, 2023 •

edited

@HaiwangYu @brettviren Similar to #250, wireplaneId and anode faceid are inconsistent for SBND APA1 (only one face, and face 1) in the codes below. Same fix as #250.
https://github.com/WireCell/wire-cell-toolkit/blob/master/sigproc/src/ROI_refinement.cxx#L2594
https://github.com/WireCell/wire-cell-toolkit/blob/master/sigproc/src/ROI_refinement.cxx#L2748
https://github.com/WireCell/wire-cell-toolkit/blob/master/sigproc/src/ROI_refinement.cxx#L2770

HaiwangYu mentioned this issue

Use IAnodePlane::which() or index #252

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment