Skip to content

[PWGLF] Add option to prevent crash on null CCDB objects #14647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
romainschotter merged 2 commits into from
Jan 27, 2026
Merged

[PWGLF] Add option to prevent crash on null CCDB objects #14647

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6985a74
Add option to prevent crash on null CCDB objects
romainschotter Jan 26, 2026
09ec1be
Fill with dummy values
romainschotter Jan 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 29 additions & 11 deletions PWGLF/TableProducer/Common/mcCentrality.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,12 +66,13 @@ struct McCentrality {
ConfigurableAxis binsMultiplicity{"binsMultiplicity", {1000, 0, 5000}, "Binning of the multiplicity axis"};
Configurable<bool> fillFt0A{"fillFt0A", false, "Fills the FT0A histogram"};
Configurable<bool> fillFt0C{"fillFt0C", false, "Fills the FT0C histogram"};
Configurable<bool> doNotCrashOnNull{"doNotCrashOnNull", false, "If ccdb object does not exist, fill with dummy values"};

HistogramRegistry histos{"histos", {}, OutputObjHandlingPolicy::AnalysisObject};

TH1F* h1dFT0M;
TH1F* h1dFT0A;
TH1F* h1dFT0C;
TH1F* h1dFT0M = nullptr;
TH1F* h1dFT0A = nullptr;
TH1F* h1dFT0C = nullptr;
// TH1F* h1dFDD;
// TH1F* h1dNTP;

Expand Down Expand Up @@ -99,13 +100,17 @@ struct McCentrality {
histos.add("FT0C/percentilevsMult", "FT0C percentile.", HistType::kTH2D, {{binsPercentile, "FT0C percentile"}, {binsMultiplicity, "FT0C mult."}});
}

TList* lOfInput;
TList* lOfInput = nullptr;
if (path.value.rfind("ccdb://", 0) == 0) { // Getting post calib. from CCDB
path.value.replace(0, 7, "");
lOfInput = ccdb->get<TList>(path);
if (!lOfInput) {
LOG(fatal) << "Could not find the calibration TList from CCDB in path " << path;
return;
if (doNotCrashOnNull) {
LOG(info) << "Could not find the calibration TList from CCDB in path " << path << ", will fill tables with dummy values";
} else {
LOG(fatal) << "Could not find the calibration TList from CCDB in path " << path;
return;
}
}
} else { // Getting post calib. from file
TFile* f = TFile::Open(path.value.c_str(), "READ");
Expand All @@ -121,11 +126,18 @@ struct McCentrality {
LOG(fatal) << "The input file " << path.value << " does not contain the TList ccdb_object";
}
}
auto getHist = [lOfInput](const char* name) -> TH1F* {
auto getHist = [this, lOfInput](const char* name) -> TH1F* {
if (!lOfInput) {
return nullptr;
}
auto hist = static_cast<TH1F*>(lOfInput->FindObject(name));
if (!hist) {
lOfInput->ls();
LOG(fatal) << "Could not open histogram " << name << " from TList";
if (this->doNotCrashOnNull) {
LOG(info) << "Could not open histogram " << name << " from TList, will fill tables with dummy values";
} else {
LOG(fatal) << "Could not open histogram " << name << " from TList";
}
}
return hist;
};
Expand All @@ -147,13 +159,19 @@ struct McCentrality {
const float nFT0M = nFT0A + nFT0C;
// const float nFV0A = mCounter.countFV0A(mcParticles);

const float valueCentFT0M = h1dFT0M->GetBinContent(h1dFT0M->FindBin(nFT0M));
float valueCentFT0M = 105.0f;
if (h1dFT0M)
valueCentFT0M = h1dFT0M->GetBinContent(h1dFT0M->FindBin(nFT0M));
if (fillFt0A) {
const float valueCentFT0A = h1dFT0M->GetBinContent(h1dFT0M->FindBin(nFT0A));
float valueCentFT0A = 105.0f;
if (h1dFT0A)
valueCentFT0A = h1dFT0A->GetBinContent(h1dFT0A->FindBin(nFT0A));
centFT0A(valueCentFT0A);
}
if (fillFt0C) {
const float valueCentFT0C = h1dFT0M->GetBinContent(h1dFT0M->FindBin(nFT0C));
float valueCentFT0C = 105.0f;
if (h1dFT0C)
valueCentFT0C = h1dFT0C->GetBinContent(h1dFT0C->FindBin(nFT0C));
centFT0C(valueCentFT0C);
}
// const float valueCentFV0A = h1dFT0M->GetBinContent(h1dFT0M->FindBin(nFV0A));
Expand Down
Loading