Skip to content

Conversation

@Luzhiyongg
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jun 19, 2025

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@github-actions github-actions bot changed the title add bool for pT ordering cut [PWGCF] add bool for pT ordering cut Jun 19, 2025
@Luzhiyongg
Copy link
Contributor Author

@victor-gonzalez Hi Victor, I'd like to ask for a quick approval for this PR beacuse I want to catch up with the auto-submission of LHC23zzh_pass5 tomorrow.

@Luzhiyongg
Copy link
Contributor Author

Also, may I ask a question for the two particle correlation.
For the same event, I have no doubt that the pT ordering (trigger pT <= associated pT) should be applied to avoid double counting.
But for mixed events, I was wondering this is still the case since we are using "strictly upper pair" event mixing.

@victor-gonzalez
Copy link
Collaborator

Also, may I ask a question for the two particle correlation. For the same event, I have no doubt that the pT ordering (trigger pT <= associated pT) should be applied to avoid double counting. But for mixed events, I was wondering this is still the case since we are using "strictly upper pair" event mixing.

Not quite sure about your question
I only see you are applying a pT ordering request which to me it is not the same as trigger pT < associated pT
If you apply the pT ordering to a mixed selecting just the upper pairs it looks to me that this is a kind of incompatible concurrent conditions: you select half of the combinations of which you will, in average, select an additional half. It sounds not good

@victor-gonzalez victor-gonzalez enabled auto-merge (squash) June 19, 2025 21:26
@victor-gonzalez
Copy link
Collaborator

@victor-gonzalez Hi Victor, I'd like to ask for a quick approval for this PR beacuse I want to catch up with the auto-submission of LHC23zzh_pass5 tomorrow.

Hope you will get it in time!

@victor-gonzalez victor-gonzalez merged commit 490435d into AliceO2Group:master Jun 19, 2025
13 checks passed
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 20, 2025
jpxrk pushed a commit to jpxrk/O2Physics that referenced this pull request Jul 16, 2025
vojmach pushed a commit to vojmach/O2Physics that referenced this pull request Jul 23, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
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.

2 participants