Skip to content

Conversation

@nmallick19
Copy link
Contributor

Following the recommendation from Physics Board on 27.01.2026,
the event selection bits for light-ion datasets has been slightly modified.

The current change replaces kIsGoodITSLayersAll with kIsGoodZvtxFT0vsPV

@victor-gonzalez
Copy link
Collaborator

This change will affect several users of this correlations framework
I agree with the incorporation of kIsGoodZvtxFT0vsPV. Now it is safe
But I don't think that in the PF it was said to remove the kIsGoodITSLayersAll selection because it quite affects the correlation functions in the bad sense
@jaelpark @jgrosseo what do you think?

@nmallick19
Copy link
Contributor Author

There exists another trigger selection which has all these three combined, i.e.,
cfgTrigger == 9 --> adds sel8 with kNoSameBunchPileup, kIsGoodZvtxFT0vsPV, and kIsGoodITSLayersAll

Thus, I thought we can have cfgTrigger == 13, exclusively for OO as suggested by PF.

@jaelpark
Copy link
Collaborator

Yes, cfgTrigger 13 was added for light ion analyses specifically. The suggested selection was indeed without kIsGoodITSLayersAll but I'm not sure how it will actually affect the correlation analyses in this case.

@victor-gonzalez
Copy link
Collaborator

There exists another trigger selection which has all these three combined, i.e., cfgTrigger == 9 --> adds sel8 with kNoSameBunchPileup, kIsGoodZvtxFT0vsPV, and kIsGoodITSLayersAll

Thus, I thought we can have cfgTrigger == 13, exclusively for OO as suggested by PF.

Who in the PF suggested to remove kIsGoodITSLayersAll for correlations in OO?

@nmallick19
Copy link
Contributor Author

There was no mention of removal of kIsGoodITSLayersAll for correlation analysis in OO by PF.
However, if I am not mistaken, one can use cfgTrigger == 9 to keep all the three selection bits, then cfgTrigger == 13 becomes redundant.

By saying so, I kept only two cases as explicitly mentioned by PF in cfgTrigger == 13
This will also enable us to compare both selection criteria, 9 vs 13

@jaelpark
Copy link
Collaborator

It was only mentioned that additional selection should be "justified". So I guess we should at least try the recommended setup.

@victor-gonzalez victor-gonzalez merged commit 2fd6543 into AliceO2Group:master Jan 30, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants