Skip to content

[Common] Add flag to save QA histograms and add event selection#15416

Merged
dsekihat merged 10 commits intoAliceO2Group:masterfrom
udmitrie:zdcextraqaflag
Mar 20, 2026
Merged

[Common] Add flag to save QA histograms and add event selection#15416
dsekihat merged 10 commits intoAliceO2Group:masterfrom
udmitrie:zdcextraqaflag

Conversation

@udmitrie
Copy link
Contributor

No description provided.

Duplicate variable for the number of ZDC towers was removed.
Added a flag cfgSaveQaHistos to control saving QA histograms and updated histogram registration accordingly.
@udmitrie udmitrie closed this Mar 16, 2026
@github-actions github-actions bot changed the title Zdcextraqaflag [Common] Zdcextraqaflag Mar 16, 2026
@udmitrie udmitrie reopened this Mar 16, 2026
@udmitrie udmitrie changed the title [Common] Zdcextraqaflag Add flag to save QA histograms Mar 16, 2026
@github-actions github-actions bot changed the title Add flag to save QA histograms [Common] Add flag to save QA histograms Mar 16, 2026
@udmitrie udmitrie marked this pull request as ready for review March 16, 2026 17:15
Rename constant kNTowers to NTowers as suggested by Vit Kucera
@udmitrie udmitrie marked this pull request as draft March 17, 2026 10:49
udmitrie and others added 3 commits March 17, 2026 14:29
Add checks isZN*Hit before filling Centroid histograms
Add event selection with Configurables.
Change default values for event selection configurations
@udmitrie udmitrie changed the title [Common] Add flag to save QA histograms [Common] Add flag to save QA histograms and add event selection Mar 17, 2026
@udmitrie udmitrie marked this pull request as ready for review March 17, 2026 13:58
@udmitrie
Copy link
Contributor Author

Dear @dsekihat, @ddobrigk, please, have a look on the suggested changes.

Exclude secondary check of z-vertex position, reuse calculated bits
@dsekihat dsekihat merged commit bfb07c5 into AliceO2Group:master Mar 20, 2026
12 checks passed
@udmitrie udmitrie deleted the zdcextraqaflag branch March 20, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants