[PWGJE] Rename Ds jet task and add THnSparse histograms#15749
[PWGJE] Rename Ds jet task and add THnSparse histograms#15749nzardosh merged 8 commits intoAliceO2Group:masterfrom
Conversation
|
O2 linter results: ❌ 1 errors, |
|
@vkucera can you check if the includes modification here is reasonable to you please? |
No, it doesn't look reasonable. |
93e1096 to
e3e2701
Compare
|
I rebased the branch on the latest master and cleaned up the changes.
The code runs correctly locally now. |
|
Hi, The CI failure is due to O2 linter errors in PWGJE/Tasks/CMakeLists.txt related to workflow naming inconsistencies (e.g. emcCellMonitor, photonIsolationQA, etc.). These are pre-existing issues and not introduced by this PR. My changes only affect the renaming of my task (jetDsSpecSubs), and I did not modify the other workflows. Please let me know how to proceed. Best, |
…e unused AOD table
f318826 to
6fbefb7
Compare
vkucera
left a comment
There was a problem hiding this comment.
@monamelop Thanks for addressing my comments.
Hi,
This PR renames the Ds jet analysis task and introduces THnSparse-based observables.
Changes:
The task jetDsSpectrumAndSubstructure was introduced during development and is not used elsewhere. This PR consolidates the final version under the name jetDsSpecSubs.
This PR has been cleaned and rebased on the latest master to include only the relevant changes.
Please let me know if you would prefer splitting the rename and the new features into separate PRs.
Thanks!