[PWGEM] PhotonMeson: Add basic QA for V0Photons#14842
[PWGEM] PhotonMeson: Add basic QA for V0Photons#14842ktf merged 1 commit intoAliceO2Group:masterfrom
Conversation
- Add `AreSelectedRunning` function to V0PhotonCuts class, which allows the usage of `EMButFlags` once at the beginning of a process function instead of checking each V0 photon inside the pair loops. - Add basic QA for V0Photons in the V0PhotonCuts class. To use it one must set the flag by using `setDoQA(true)` AND providing a pointer to the histogram registry of the main task. To initilize the histograms one needs to call `fV0PhotonCut.addQAHistograms(®istry)` in the init function of the task. For proper usage one needs to use `AreSelectedRunning`. It is possible to do it with the single V0 check function IsSelected(), however, not all histograms will be properly filled in that case. - Fix basic QA inside EMCPhotonCuts
alibuild
left a comment
There was a problem hiding this comment.
Auto-approving on behalf of @mhemmer-cern.
|
The error on macOS seems to be genuine. Can you have a look? |
I just checked and it is not genuine, the variable is used: It was modified last by this PR #14760 |
|
@mhemmer-cern actually it was and I fixed it in 9a2575e. I think the one you refer to is on a different line. Anyway, this is unrelated to this PR. Merging. |
Ah okay, then there must have been multiple declarations of that variable at some point before or something. Thanks for checking and merging! |
AreSelectedRunningfunction to V0PhotonCuts class, which allows the usage ofEMButFlagsonce at the beginning of a process function instead of checking each V0 photon inside the pair loops.setDoQA(true)AND providing a pointer to the histogram registry of the main task. To initilize the histograms one needs to callfV0PhotonCut.addQAHistograms(®istry)in the init function of the task. For proper usage one needs to useAreSelectedRunning. It is possible to do it with the single V0 check function IsSelected(), however, not all histograms will be properly filled in that case.