[PWGHF]: Fix search of mothers to work also at partonic level#939
Merged
Conversation
fgrosa
requested review from
a team,
alibuild,
fcolamar,
ginnocen,
iarsene,
jgrosseo,
ktf and
vkucera
as code owners
fgrosa
changed the title
Collaborator
|
Please add the PWGHF: prefix in the commit title when your PR has only one commit as the PR title is ignored in such a case. |
Collaborator
|
Thanks for the fix. Can you please add comments in the code to help understand how you use the new variables? |
Collaborator
Author
Sure, I added them! Let me know if something is still unclear |
Collaborator
Thanks |
vkucera
approved these changes
Jul 5, 2022
Collaborator
Collaborator
Author
Collaborator
jgrosseo
approved these changes
Jul 6, 2022
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
3 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.
This PR fixes the search of the mother particles, in particular when needed at partonic level (i.e. non-prompt tagging for HF analyses). In the previous version, only the first element of the mothers vector was checked, however most of the times the beauty quark is not in the first element. This was the origin of the issue related to the non-prompt tagging found in O2-2761, now fixed with this PR.