[PWGHF] Fix missing McCollisions and refactor MC histograms into subdirectories in taskLcToK0sP.cxx#16541
[PWGHF] Fix missing McCollisions and refactor MC histograms into subdirectories in taskLcToK0sP.cxx#16541dongfangqingxin wants to merge 4 commits into
Conversation
|
O2 linter results: ❌ 0 errors, |
|
Error while checking build/O2Physics/o2 for 4ac3499 at 2026-06-05 18:09: Full log here. |
|
Error while checking build/O2Physics/staging for 4ac3499 at 2026-06-05 18:13: Full log here. |
|
@dongfangqingxin Did you check for warnings in your compilation? |
Hi @vkucera Thanks for the comments! I used a script that only catches errors, not warnings and I did not check log manually — my bad. I'm going to drop the extractCandVars template and just repeat the code instead. |
Why do you want to drop it? |
Since both alibuild and the O2 linter were complaining about the template I and figured removing it was the quickest fix |
…l/prompt/nonprompt/background subdirectories - Add missing McCollisions dependency - Reorganise MC/Rec and MC/Gen histograms into signal/prompt/nonprompt/background subdirectories - Code clean up
O2 linter did not complain about the template and the compilation warning was about a narrowing |
|
Do not force-push. You just overwrote the commit that I have reviewed. It's explicitly forbidden in the instructions. |
@vkucera Sorry about the force-push, won't happen again. I'll add new commits from now on. I kept the template and all your comments have been addressed in the latest version, except the one about KDecayChannelLcToK0sP — do you have any suggestion for it? |
Yes, your are right! I renamed it to HfCandVars to fix the O2 linter error, and changed the member types from float to double to fix the narrowing conversion warning. |
vkucera
left a comment
There was a problem hiding this comment.
Thanks @dongfangqingxin
|
taskLcToK0sP.cxx was Updated in this PR