[PWGHF] Add hadron and electron PID cuts in HFE–hadron correlation#14767
Open
rashigupt wants to merge 18 commits intoAliceO2Group:masterfrom
Open
[PWGHF] Add hadron and electron PID cuts in HFE–hadron correlation#14767rashigupt wants to merge 18 commits intoAliceO2Group:masterfrom
rashigupt wants to merge 18 commits intoAliceO2Group:masterfrom
Conversation
Two separate columns are included in the analysis table: one corresponding to the LSmass selection and the other to the ULmass selection, and the common mass column has been removed.
We define the mass column in the analysis by splitting it into two separate columns, LSmass and ULmass.
…ass. We use the mass column correctly by separating it into LSmass and ULmass.
Add PID cut for find the efficiency
Add PID cut for find Hadron efficiency
rashigupt
requested review from
NicoleBastid,
alibuild,
apalasciano,
deepathoms,
fcatalan92,
fcolamar,
fgrosa,
hahassan7,
jpxrk,
mfaggin,
mmazzilli,
vkucera and
zhangbiao-phy
as code owners
rashigupt
requested review from
gluparel,
singhra1994 and
stefanopolitano
as code owners
github-actions
bot
changed the title
Collaborator
|
Please do not write a full story of your motivations in the PR title. You can give all the details in the PR description. The PR title is supposed to be a concise summary of what changed and where. |
Contributor
Author
|
Thank you for the suggestion. I have updated the PR title to make it concise and focused on the actual change. |
Collaborator
The title has not changed. |
rashigupt
changed the title
gluparel
previously approved these changes
Feb 4, 2026
vkucera
requested changes
Feb 4, 2026
| // Electron-hadron Correlation | ||
| template <typename TracksType, typename ElectronType, typename CollisionType, typename BcType> | ||
| void fillCorrelation(CollisionType const& collision, ElectronType const& electrons, TracksType const& tracks, BcType const&) | ||
| template <bool isMc, typename TracksType, typename ElectronType, typename CollisionType, typename BcType, typename McParticles> |
Collaborator
There was a problem hiding this comment.
Suggested change
| template <bool isMc, typename TracksType, typename ElectronType, typename CollisionType, typename BcType, typename McParticles> | |
| template <bool IsMc, typename TracksType, typename ElectronType, typename CollisionType, typename BcType, typename McParticles> |
McParticles is an existing table type. Do not use it as a template parameter.
Contributor
Author
There was a problem hiding this comment.
Thankyou for the suggestion
Comment on lines
230
to
244
| if (std::abs(mcParticle.pdgCode()) != kElectron) { | ||
| registry.fill(HIST("hMCRecptEleHadron"), hTrack.pt()); | ||
| } | ||
| if (std::abs(mcParticle.pdgCode()) != kMuonMinus) { | ||
| registry.fill(HIST("hMCRecptMuonHadron"), hTrack.pt()); | ||
| } | ||
| if (std::abs(mcParticle.pdgCode()) != kPiPlus) { | ||
| registry.fill(HIST("hMCRecptPionHadron"), hTrack.pt()); | ||
| } | ||
| if (std::abs(mcParticle.pdgCode()) != kKPlus) { | ||
| registry.fill(HIST("hMCRecptKaonHadron"), hTrack.pt()); | ||
| } | ||
| if (std::abs(mcParticle.pdgCode()) != kProton) { | ||
| registry.fill(HIST("hMCRecptProtonHadron"), hTrack.pt()); | ||
| } |
Collaborator
There was a problem hiding this comment.
These conditions are mutually exclusive. Use else if or a switch statement.
Contributor
Author
There was a problem hiding this comment.
Thankyou for the guidance
Comment on lines
470
to
479
| if (std::abs(particleMc.pdgCode()) != kElectron) { | ||
| registry.fill(HIST("hMCgenptEleHadron"), particleMc.pt()); | ||
| } | ||
| if (std::abs(particleMc.pdgCode()) != kMuonMinus) { | ||
| registry.fill(HIST("hMCgenptMuonHadron"), particleMc.pt()); | ||
| } | ||
| if (std::abs(particleMc.pdgCode()) != kPiPlus) { | ||
| registry.fill(HIST("hMCgenptPionHadron"), particleMc.pt()); | ||
| } | ||
| if (std::abs(particleMc.pdgCode()) != kKPlus) { |
Contributor
Author
There was a problem hiding this comment.
Thank you for the guidance.
vkucera
reviewed
Feb 4, 2026
Comment on lines
230
to
232
| if (std::abs(mcParticle.pdgCode()) != kElectron) { | ||
| registry.fill(HIST("hMCRecptEleHadron"), hTrack.pt()); | ||
| } |
Collaborator
There was a problem hiding this comment.
Why do you fill the histogram with particles which are not electrons?
Contributor
Author
There was a problem hiding this comment.
Apologies for the mistake. Thank you for pointing it out!
…ter PID for correlation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Electron and hadron efficiencies are evaluated by applying PID cuts at both the MC generation and reconstruction levels: