-
Notifications
You must be signed in to change notification settings - Fork 483
MCH add cut random fraction setting for the cases with and without ITS #14774
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.
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
MCH add cut random fraction setting for the cases with and without ITS #14774
Conversation
Two new environment variables are introduced to provide the fraction of rejected MCH events separately for the cases where the selection based on ITS IRframes is either enabled (CUT_RANDOM_FRACTION_MCH_WITH_ITS) or disabled (CUT_RANDOM_FRACTION_MCH_NO_ITS). The selection based on ITS is enabled by default in SYNC processing for physics runs where ITS is included in the data taking. It can be however explicitely disabled by adding MCH_DISABLE_ITS_IRFRAMES_SELECTION=1 to the custom environment variables. In this case, CUT_RANDOM_FRACTION_MCH_NO_ITS is used. The CUT_RANDOM_FRACTION_MCH variable, as well as the two new ones, can still be overridden by setting them explicitly in the shell environment. Moreover, the ITS selection is applied whenever MCH_DISABLE_ITS_IRFRAMES_SELECTION is undefined, therefore the changes are fully backward compatible.
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
@davidrohr @ehellbar this is an update for PR #14028 (which I did not realize was closed without merging), addressing your comments and also adding an optional variable for explicitly disabling the ITS selection in physics runs. In particular, the explicit disabling will be needed to investigate the beam background in the first part of the Pb-Pb run this year, since the background effects are usually suppressed in SYNC when requiring ITS vertices. We would therefore need to deploy it to P2 before the start of the Pb-Pb run, ideally this week. This was already discussed and agreed with RC. Thanks a lot in advance! |
|
Error while checking build/O2/fullCI_slc9 for 7cd2a0c at 2025-10-28 13:51: Full log here. |
|
@ehellbar the CI is green, can we proceed with the merging, or is there something else to check/modify? Thanks! |
Two new environment variables are introduced to provide the fraction of rejected MCH events separately for the cases where the selection based on ITS IRframes is either enabled (CUT_RANDOM_FRACTION_MCH_WITH_ITS) or disabled (CUT_RANDOM_FRACTION_MCH_NO_ITS).
The selection based on ITS is enabled by default in SYNC processing for physics runs where ITS is included in the data taking. It can be however explicitely disabled by adding MCH_DISABLE_ITS_IRFRAMES_SELECTION=1 to the custom environment variables. In this case, CUT_RANDOM_FRACTION_MCH_NO_ITS is used.
The CUT_RANDOM_FRACTION_MCH variable, as well as the two new ones, can still be overridden by setting them explicitly in the shell environment. Moreover, the ITS selection is applied whenever MCH_DISABLE_ITS_IRFRAMES_SELECTION is undefined, therefore the changes are fully backward compatible.