PWGHF: Add solid table to be written to file #4910
Merged
Conversation
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
njacazio
force-pushed
the
nj-hftreewriter
branch
8 times, most recently
from
9b85be2 to
469ca85
Compare
njacazio
force-pushed
the
nj-hftreewriter
branch
from
469ca85 to
a386849
Compare
ginnocen
changed the title
Collaborator
|
@njacazio do you have also an equivalent PR for the Lc->pKpi? |
njacazio
force-pushed
the
nj-hftreewriter
branch
from
a386849 to
f2afe0e
Compare
Collaborator
Author
|
@ginnocen now the PR has also Lc ! Do you think it is worth to have it merged in O2? |
njacazio
force-pushed
the
nj-hftreewriter
branch
from
f2afe0e to
82e5981
Compare
Collaborator
Author
|
@ginnocen if it's ok with you I would merge this |
vkucera
reviewed
Jan 15, 2021
ginnocen
previously approved these changes
Jan 16, 2021
ginnocen
reviewed
Jan 16, 2021
Collaborator
|
@DelloStritto please have a look too |
Collaborator
|
Hi @njacazio I am here referring to the fact that we are filling the candidates only once with the mass of the D0, while in principle we know that both mass hypothesis should be considered, D0 and D0bar. We need to think about how to do this in the cleanest way.. |
Collaborator
Author
|
That's fine I will add the D0bar as well |
Collaborator
|
lets have a look at it with the others after the WP414 |
vkucera
reviewed
Jan 18, 2021
njacazio
commented
Jan 20, 2021
njacazio
force-pushed
the
nj-hftreewriter
branch
from
e22298b to
2ea6403
Compare
vkucera
requested changes
Jan 21, 2021
njacazio
force-pushed
the
nj-hftreewriter
branch
from
3b1605b to
0c0ad42
Compare
- Add full table to be written to file - Add task dedicated to 2 prong tree writing - Add task dedicated to 3 prong tree writing - Use naming of tasks as in HFSecondaryVertex
njacazio
force-pushed
the
nj-hftreewriter
branch
from
0c0ad42 to
f0b2bef
Compare
ginnocen
approved these changes
Jan 22, 2021
| candidate.index2_as<aod::BigTracksPID>().tpcNSigmaKa(), | ||
| candidate.index2_as<aod::BigTracksPID>().tofNSigmaPi(), | ||
| candidate.index2_as<aod::BigTracksPID>().tofNSigmaKa(), | ||
| InvMassLcpKpi(candidate), |
Collaborator
There was a problem hiding this comment.
same here, @njacazio @nzardosh @vkucera @DelloStritto are we here only filling the pKpi?
ginnocen
added a commit
to ginnocen/AliceO2
that referenced
this pull request
Jan 23, 2021
This reverts commit 9b2a233.
jgrosseo
pushed a commit
that referenced
this pull request
Jan 23, 2021
EmilGorm
pushed a commit
to EmilGorm/AliceO2
that referenced
this pull request
Nov 22, 2021
- Add full table to be written to file - Add task dedicated to 2 prong tree writing - Add task dedicated to 3 prong tree writing - Use naming of tasks as in HFSecondaryVertex
EmilGorm
pushed a commit
to EmilGorm/AliceO2
that referenced
this pull request
Nov 22, 2021
…" (AliceO2Group#5270) This reverts commit 9b2a233.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 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.
@ginnocen