Skip to content

[Common] Add option to directly fetch centrality calibration histograms in centrality study#16476

Open
jesgum wants to merge 11 commits into
AliceO2Group:masterfrom
jesgum:master
Open

[Common] Add option to directly fetch centrality calibration histograms in centrality study#16476
jesgum wants to merge 11 commits into
AliceO2Group:masterfrom
jesgum:master

Conversation

@jesgum
Copy link
Copy Markdown
Collaborator

@jesgum jesgum commented May 31, 2026

@github-actions github-actions Bot changed the title Add IR vs Centrality histograms to centrality study [Common] Add IR vs Centrality histograms to centrality study May 31, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 31, 2026

O2 linter results: ❌ 2 errors, ⚠️ 6 warnings, 🔕 0 disabled

Please consider the following formatting changes to AliceO2Group#16476
@alibuild
Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 432a81c at 2026-05-31 13:44:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:874:13: error: expected primary-expression before 'constexpr'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:874:13: error: expected ')' before 'constexpr'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:875:71: error: 'const struct o2::soa::Table<o2::aod::Hash<519320931>, o2::aod::Hash<1837103245>, o2::aod::Hash<519320931>, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >::TableIteratorBase<o2::soa::DefaultIndexPolicy, o2::soa::Join<o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >' has no member named 'centFT0C'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:877:85: error: 'const struct o2::soa::Table<o2::aod::Hash<519320931>, o2::aod::Hash<1837103245>, o2::aod::Hash<519320931>, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >::TableIteratorBase<o2::soa::DefaultIndexPolicy, o2::soa::Join<o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >' has no member named 'centFT0C'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:875:71: error: 'const struct o2::soa::Table<o2::aod::Hash<519320931>, o2::aod::Hash<1837103245>, o2::aod::Hash<519320931>, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >::TableIteratorBase<o2::soa::DefaultIndexPolicy, o2::soa::Join<o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >' has no member named 'centFT0C'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/Common/Tasks/centralityStudy.cxx:877:85: error: 'const struct o2::soa::Table<o2::aod::Hash<519320931>, o2::aod::Hash<1837103245>, o2::aod::Hash<519320931>, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >::TableIteratorBase<o2::soa::DefaultIndexPolicy, o2::soa::Join<o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >, o2::soa::Join<o2::soa::Table<o2::aod::Hash<558354758>, o2::aod::Hash<620079319>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1535033157>, o2::aod::Hash<3179724803>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<4055870789>, o2::aod::Hash<3173777681>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1713421420>, o2::aod::Hash<4278781548>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2437771125>, o2::aod::Hash<2610283391>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2807856041>, o2::aod::Hash<348540052>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<1483628689>, o2::aod::Hash<1010550289>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3139039590>, o2::aod::Hash<414044937>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3595307350>, o2::aod::Hash<2823750881>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2006819746>, o2::aod::Hash<1709953559>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1282963412>, o2::aod::Hash<2334075379>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<1769045462>, o2::aod::Hash<1063043080>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3943458539>, o2::aod::Hash<2000676492>, o2::aod::Hash<2286545062> > >' has no member named 'centFT0C'
ninja: build stopped: subcommand failed.

Full log here.

@jesgum jesgum marked this pull request as draft June 1, 2026 10:02
@jesgum jesgum changed the title [Common] Add IR vs Centrality histograms to centrality study [Common] Add option to directly fetch centrality calibration histograms Jun 1, 2026
@jesgum jesgum changed the title [Common] Add option to directly fetch centrality calibration histograms [Common] Add option to directly fetch centrality calibration histograms in centrality study Jun 1, 2026
@jesgum jesgum marked this pull request as ready for review June 1, 2026 12:19
Configurable<std::string> irSource{"irSource", "ZNC hadronic", "Source of the interaction rate: (Recommended: pp --> T0VTX, Pb-Pb --> ZNC hadronic)"};
Configurable<bool> irCrashOnNull{"irCrashOnNull", false, "Flag to avoid CTP RateFetcher crash."};
Configurable<bool> fetchCentralityCalibration{"fetchCentralityCalibration", false, "Flag to fetch the centrality calibration within the task instead of the centrality table"};
} ccdb;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a very misleading name, especially in combination with the renamed member mCcdb.

mCcdb->setURL(ccdb.ccdbURL);
// mCcdb->setCaching(true);
// mCcdb->setLocalObjectValidityChecking();
mCcdb->setFatalWhenNull(false);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefix m is reserved for members of classes.

// ccdb->setCaching(true);
// ccdb->setLocalObjectValidityChecking();
ccdb->setFatalWhenNull(false);
hCentralityFV0A = nullptr;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you set it to nullptr again?

Comment on lines 229 to 237
hCalibObjects = nullptr;
hVtxZFV0A = nullptr;
hVtxZFT0A = nullptr;
hVtxZFT0C = nullptr;
hVtxZNTracks = nullptr;
hVtxZNGlobals = nullptr;
hVtxZMFT = nullptr;
hVtxZFDDA = nullptr;
hVtxZFDDC = nullptr;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be done in the declaration.

hCentralityMFT = dynamic_cast<TH1*>(hCentralityObjects->FindObject("hCalibZeqMFT"));

// won't capture null pointers -> explicitly check for those when attempting to evaluate
auto reportSuccess = [](TH1* hist, std::string name) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't pass string by value.

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