Skip to content
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

Common part pdg and primary has wrong value for Pandora #94

Closed
noeroy opened this issue Jan 24, 2025 · 2 comments
Closed

Common part pdg and primary has wrong value for Pandora #94

noeroy opened this issue Jan 24, 2025 · 2 comments
Assignees

Comments

@noeroy
Copy link
Contributor

noeroy commented Jan 24, 2025

Issue in the definition of common.part.pdg, common.part.primary in Pandora reco

https://github.com/DUNE/ND_CAFMaker/blob/main/src/reco/PandoraLArRecoNDBranchFiller.cxx#L412-L417
https://github.com/DUNE/ND_CAFMaker/blob/main/src/reco/PandoraLArRecoNDBranchFiller.cxx#L473-L478

those should be based on reco quantities and not truth quantities in that branch.
Primary in that sense is "linked to the vertex or not"
PDG in that sense is "whatever pdg reconstructed for that particle" (most likely 11 for electron showers, 13 for muon tracks).
https://github.com/DUNE/duneanaobj/blob/main/duneanaobj/StandardRecord/SRRecoParticle.h#L26-L28

Answer from @jback08
We will need to add the reconstruction isPrimary and PDG hypothesis variables to the Pandora ROOT output, which is underway with LArRecoND PandoraPFA/LArRecoND#24. When this has been merged, we can then update the Pandora ND CAF maker to use these new ntuple variables to set these common values correctly.

@jback08
Copy link
Member

jback08 commented Mar 6, 2025

A PR request has been made for this: #95

@jback08
Copy link
Member

jback08 commented Mar 7, 2025

Closing this, since the code has been updated in merged PR #95.

@jback08 jback08 closed this as completed Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants