From 74e175df68d92adf8c52bdc3fbd9b128f2139b53 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=B2=20Jacazio?= Date: Thu, 8 Jan 2026 15:51:15 +0100 Subject: [PATCH 1/3] Update onTheFlyTrackerPid.cxx --- ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx index a8bf82a3de1..9e5da11d637 100644 --- a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx +++ b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx @@ -142,8 +142,8 @@ class ToTLUT } if (strncmp(filename.c_str(), "ccdb:", 5) == 0) { // Check if filename starts with "ccdb:" const std::string basePath = std::string(filename).substr(5); - const std::string outPath = "/tmp/ToTLUTs/" + basePath; - const std::string localFilename = outPath + "/snapshot.root"; + const std::string outPath = "/tmp/ToTLUTs/"; + const std::string localFilename = outPath + basePath + "/snapshot.root"; std::ifstream checkFile(localFilename); if (!checkFile.is_open()) { // File is not found, need to download it from CCDB if (!mCcdbManager) { @@ -153,7 +153,7 @@ class ToTLUT mCcdbManager->getCCDBAccessor().retrieveBlob(basePath, outPath, metadata, 1); std::ifstream testFile(localFilename); if (!testFile.is_open()) { - LOG(fatal) << "Could not find downloaded CCDB file for PDG " << pdg; + LOG(fatal) << "Could not find downloaded CCDB file for PDG " << pdg << " in file "<< localFilename; return false; } testFile.close(); From 171685f121e40a4f9fd5034df4f1551e964735a3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=B2=20Jacazio?= Date: Thu, 8 Jan 2026 16:13:36 +0100 Subject: [PATCH 2/3] Update onTheFlyTrackerPid.cxx --- .../TableProducer/OTF/onTheFlyTrackerPid.cxx | 27 ++++++++++--------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx index 9e5da11d637..adb68c87013 100644 --- a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx +++ b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx @@ -140,17 +140,18 @@ class ToTLUT LOG(warning) << "Provided filename is empty for PDG " << pdg; return false; } - if (strncmp(filename.c_str(), "ccdb:", 5) == 0) { // Check if filename starts with "ccdb:" - const std::string basePath = std::string(filename).substr(5); + if (filename.rfind("ccdb:", 0) == 0) { // Check if filename starts with "ccdb:" + std::string ccdbPath = filename.substr(5); // remove "ccdb:" prefix const std::string outPath = "/tmp/ToTLUTs/"; - const std::string localFilename = outPath + basePath + "/snapshot.root"; + const std::string localFilename = outPath + ccdbPath + "/snapshot.root"; std::ifstream checkFile(localFilename); if (!checkFile.is_open()) { // File is not found, need to download it from CCDB if (!mCcdbManager) { LOG(fatal) << "CCDB manager not set. Please set it before loading LUT from CCDB."; } std::map metadata; - mCcdbManager->getCCDBAccessor().retrieveBlob(basePath, outPath, metadata, 1); + LOG(info) << "Retrieving " << localFilename << " from CCDB path: " << ccdbPath; + mCcdbManager->getCCDBAccessor().retrieveBlob(ccdbPath, outPath, metadata, 1); std::ifstream testFile(localFilename); if (!testFile.is_open()) { LOG(fatal) << "Could not find downloaded CCDB file for PDG " << pdg << " in file "<< localFilename; @@ -396,15 +397,15 @@ struct OnTheFlyTrackerPid { HistogramRegistry histos{"histos", {}, OutputObjHandlingPolicy::AnalysisObject}; - Configurable lutTotEl{"lutTotEl", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_11/", "ToT LUT for electrons"}; - Configurable lutTotMu{"lutTotMu", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_13/", "ToT LUT for muons"}; - Configurable lutTotPi{"lutTotPi", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_211/", "ToT LUT for pions"}; - Configurable lutTotKa{"lutTotKa", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_321/", "ToT LUT for kaons"}; - Configurable lutTotPr{"lutTotPr", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_2212/", "ToT LUT for protons"}; - Configurable lutTotDe{"lutTotDe", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000010020/", "ToT LUT for deuteron"}; - Configurable lutTotTr{"lutTotTr", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000010030/", "ToT LUT for triton"}; - Configurable lutTotHe{"lutTotHe", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000020030/", "ToT LUT for helium-3"}; - Configurable lutTotAl{"lutTotAl", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000020040/", "ToT LUT for alphas"}; + Configurable lutTotEl{"lutTotEl", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_11", "ToT LUT for electrons"}; + Configurable lutTotMu{"lutTotMu", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_13", "ToT LUT for muons"}; + Configurable lutTotPi{"lutTotPi", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_211", "ToT LUT for pions"}; + Configurable lutTotKa{"lutTotKa", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_321", "ToT LUT for kaons"}; + Configurable lutTotPr{"lutTotPr", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_2212", "ToT LUT for protons"}; + Configurable lutTotDe{"lutTotDe", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000010020", "ToT LUT for deuteron"}; + Configurable lutTotTr{"lutTotTr", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000010030", "ToT LUT for triton"}; + Configurable lutTotHe{"lutTotHe", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000020030", "ToT LUT for helium-3"}; + Configurable lutTotAl{"lutTotAl", "ccdb:Users/h/hfribert/ToT_LUTs/PDG_1000020040", "ToT LUT for alphas"}; Configurable maxBarrelLayers{"maxBarrelLayers", 11, "Maximum number of barrel layers"}; Configurable numLogBins{"numLogBins", 200, "Number of logarithmic momentum bins"}; From cbe5f8adb802db3d1ced3c3e6d2559fbb84f3f8b Mon Sep 17 00:00:00 2001 From: ALICE Builder Date: Fri, 9 Jan 2026 08:54:34 +0100 Subject: [PATCH 3/3] Please consider the following formatting changes to #14434 (#14435) --- ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx index adb68c87013..f5ab4c78495 100644 --- a/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx +++ b/ALICE3/TableProducer/OTF/onTheFlyTrackerPid.cxx @@ -140,7 +140,7 @@ class ToTLUT LOG(warning) << "Provided filename is empty for PDG " << pdg; return false; } - if (filename.rfind("ccdb:", 0) == 0) { // Check if filename starts with "ccdb:" + if (filename.rfind("ccdb:", 0) == 0) { // Check if filename starts with "ccdb:" std::string ccdbPath = filename.substr(5); // remove "ccdb:" prefix const std::string outPath = "/tmp/ToTLUTs/"; const std::string localFilename = outPath + ccdbPath + "/snapshot.root"; @@ -154,7 +154,7 @@ class ToTLUT mCcdbManager->getCCDBAccessor().retrieveBlob(ccdbPath, outPath, metadata, 1); std::ifstream testFile(localFilename); if (!testFile.is_open()) { - LOG(fatal) << "Could not find downloaded CCDB file for PDG " << pdg << " in file "<< localFilename; + LOG(fatal) << "Could not find downloaded CCDB file for PDG " << pdg << " in file " << localFilename; return false; } testFile.close();