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] To fix mistake for getParticleOrigin #8661

Closed
wants to merge 1 commit into from

Conversation

hanseopark
Copy link
Contributor

I have mistaken to write this function.
The direction I want to go is that if the mother does not have a parton, then I should check the particle's pdgcode.
Because if a hadron does not have a parton as its mother, but it is not a simple break, we can assign the origin of the particle based on the pdgcode of the hadron.
if it breaks, it move to OriginType::None

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 5ad2dd8 at 2024-12-12 14:07:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8661-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:17: error: 'TracksQA_002' is not a member of 'o2::aod'; did you mean 'TracksQA_001'?
/sw/SOURCES/O2Physics/8661-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:29: error: template argument 1 is invalid
/sw/SOURCES/O2Physics/8661-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:25:32: error: 'tracksQA_000' was not declared in this scope; did you mean 'tracksQA_002'?
/sw/SOURCES/O2Physics/8661-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:59:32: error: 'tracksQA_001' was not declared in this scope; did you mean 'tracksQA_002'?
ninja: build stopped: subcommand failed.

Full log here.

Copy link

This PR has not been updated in the last 30 days. Is it still needed? Unless further action is taken, it will be closed in 5 days.

@github-actions github-actions bot added the stale label Jan 12, 2025
@github-actions github-actions bot closed this Jan 18, 2025
@hanseopark
Copy link
Contributor Author

I would like to check this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants