[PWGCF] Option to reject parts of the FT0 detectors#14219
[PWGCF] Option to reject parts of the FT0 detectors#14219MartijnLaarhoven wants to merge 7 commits intoAliceO2Group:masterfrom
Conversation
Added option to reject the inside or the outside of the detector ring of FT0
Added option to reject the inside or the outside of the detector ring of FT0
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
making the rejection of the rejection of part of the detector more distinctive
MartijnLaarhoven
requested review from
alibuild,
ariedel-cern,
glromane,
lauraser,
lgraczykCern,
otonvd,
prchakra,
shouqiye,
victor-gonzalez and
zchochul
as code owners
|
O2 linter results: ❌ 0 errors, |
github-actions
bot
changed the title
There was a problem hiding this comment.
Pull request overview
This PR adds configuration options to selectively reject inner or outer ring channels of the FT0 detector by zeroing their amplitudes during analysis. This allows for more granular control over which parts of the FT0A and FT0C detectors contribute to correlation measurements.
Key Changes:
- Added two boolean configuration options:
cfgRejectOutsideDetectorsandcfgRejectInsideDetectorsto control which FT0 detector rings to reject - Introduced
DetectorChannelsenum defining the channel ID ranges for inner (0-31, 96-143) and outer (32-95, 144-207) rings of FT0A and FT0C detectors - Implemented amplitude zeroing logic in the
getChannelfunction that checks channel IDs against the enum ranges and sets amplitude to 0 when rejection conditions are met
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| O2_DEFINE_CONFIGURABLE(cfgRejectOutsideDetectors, bool, false, "Rejection of outside ring events of the FT0 detector") | ||
| O2_DEFINE_CONFIGURABLE(cfgRejectInsideDetectors, bool, false, "Rejection of inside ring events of the FT0 detector") |
There was a problem hiding this comment.
If both cfgRejectInsideDetectors and cfgRejectOutsideDetectors are enabled simultaneously, all FT0 detector channels will be zeroed out. Consider adding validation to prevent both flags from being true at the same time, or documenting this behavior if it's intentional.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 participant
Option to reject parts of the FT0 detectors