[PWGEM] add new pairQC Task + clean up photonhbt - #17984
Conversation
|
O2 linter results: ❌ 0 errors, |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
mhemmer-cern
left a comment
There was a problem hiding this comment.
Please check the CMakeList changes again and while you are at it take a look at the code-check results to at least fix the introduced errors please.
| #include <cmath> | ||
| #include <cstdint> | ||
| #include <cstdlib> | ||
| #include <ranges> |
There was a problem hiding this comment.
ranges is actually used in this header file for std::ranges::input_range. Could you add it back in please
mhemmer-cern
left a comment
There was a problem hiding this comment.
Looks good to me now
|
Since code check errors found here were fixed I will enable auto-merge, so this can make it into tomorrows tag without needing to wait for the builds |
|
Error while checking build/O2Physics/code-check for f7f1685 at 2026-09-20 21:53: Full log here. |
|
Error while checking build/O2Physics/code-check for 2d9c546 at 2026-09-20 22:05: Full log here. |
New task pairQcTask: analysis of pairs of V0 photon candidates (same event only). The following analyses are possible:
(Utils/AnalyticV0.h), and the veto derived from them
added new functions into MCUtilities and PairUtilities, so that multiple tasks can use those functions to analyse certain reconstruction artifacts.
Moved the PairQA out from photonhbt, so that only the correlation function analysis and the MC analysis stays there. Added a new Mixing with V0 Legs only, based on V0Leg information and V0LegXYZ information.