Uh oh!
There was an error while loading. Please reload this page.
PWGCF: add triton and He3 PID to femtodream data model - #8123
Conversation
victor-gonzalez
left a comment
There was a problem hiding this comment.
Anton, please, have a look at my only comment
Is that everything that is needed to incorporate He3 to the derived data?
| case 1000010020: // Deuteron | ||
| pidTPC = part.tpcNSigmaDe(); | ||
| pidTOF = part.tofNSigmaDe(); | ||
| break; | ||
| case 1000010030: // Triton | ||
| pidTPC = part.tpcNSigmaTr(); | ||
| pidTOF = part.tofNSigmaTr(); | ||
| break; | ||
| case 1000020030: // Helium3 |
There was a problem hiding this comment.
Could we move to use named constants as per
https://github.com/AliceO2Group/AliceO2/blob/dev/Common/Constants/include/CommonConstants/PhysicsConstants.h
it will be less error prone and no comments are needed
As it is commented there, the previous file expands the definitions in <TPDGCode_8.h> as visible in https://root.cern/doc/master/TPDGCode_8h.html
There was a problem hiding this comment.
Hi Victor,
yes. I just need to add the PID to the extra tables and some additional histograms. The framework which calculates the bitmask did already support all possible track PIDs.
Thanks for the comment. I swich the PDG codes to the name constants.
51e07e9 to
3260357Compareariedel-cern
commented
Oct 25, 2024
Hi @mfaggin@fgrosa @1994ra , |
fgrosa
left a comment
There was a problem hiding this comment.
Hi @AntonRiedel I am approving it not to block your work, but I think it would be better to have joinable tables for the PID hypotheses, not to have many unused dummy columns when not needed
victor-gonzalez
commented
Oct 28, 2024
@ariedel-cern, @fgrosa made a good point! |
Add triton and He3 PID to femtodream data model.