[PWGHF] Update tree creator for Xic0Omegac0#16421
Conversation
|
O2 linter results: ❌ 0 errors, |
Please consider the following formatting changes to AliceO2Group#16421
|
You are repeating the same block 11 times! |
… into updateworkflow
I've written a helper function to fill in particle table to enhance code maintainence. |
…to updateworkflow
|
Hi @vkucera, I've tried compiling O2Physics with this PR and found no issues. |
You cannot use floating-point values as template arguments. Why would you do it anyway? The mass value does not change the code. |
I wasn't aware of that. Thanks for pointing that out. |
I understand you want the function to be able to digest different mass values but a simple function argument should work fine, no? Template arguments are for generating different versions of the code which does not seem to be the case here. |
Yes, I fully agree with you. It was my mistake to assign mass arguement to template arguement. Thanks again :) |
|
Dear @vkucera, |
Thanks, looks good to me. |
|
Dear @zhangbiao-phy, I've prepared a PR to update tree creator of Xic0Omegac0 workflow. |
Modified
treeCreatorToXiPiQa.cxxto support saving an output table with generated-particle information.