From 86376980cc66f3feffe939da6fa91522c0c00be2 Mon Sep 17 00:00:00 2001 From: cnkoster Date: Fri, 25 Jul 2025 16:06:43 +0200 Subject: [PATCH 1/6] Add options for 2d qa track histos for centrality dependence --- PWGCF/Flow/TableProducer/zdcQVectors.cxx | 1 + 1 file changed, 1 insertion(+) diff --git a/PWGCF/Flow/TableProducer/zdcQVectors.cxx b/PWGCF/Flow/TableProducer/zdcQVectors.cxx index c748b1f51ec..6048701e258 100644 --- a/PWGCF/Flow/TableProducer/zdcQVectors.cxx +++ b/PWGCF/Flow/TableProducer/zdcQVectors.cxx @@ -334,6 +334,7 @@ struct ZdcQVectors { } registry.fill(HIST("hEventCount"), evSel_kNoSameBunchPileup); } + if (cfgEvSelsIsGoodZvtxFT0vsPV) { if (!collision.selection_bit(o2::aod::evsel::kIsGoodZvtxFT0vsPV)) { // removes collisions with large differences between z of PV by tracks and z of PV from FT0 A-C time difference From d8a7fcfaa802c30cc76c9cff9ea9b1ffdbd01b65 Mon Sep 17 00:00:00 2001 From: cnkoster Date: Fri, 25 Jul 2025 16:09:16 +0200 Subject: [PATCH 2/6] Added wrong file --- PWGCF/Flow/TableProducer/zdcQVectors.cxx | 1 - 1 file changed, 1 deletion(-) diff --git a/PWGCF/Flow/TableProducer/zdcQVectors.cxx b/PWGCF/Flow/TableProducer/zdcQVectors.cxx index 6048701e258..c748b1f51ec 100644 --- a/PWGCF/Flow/TableProducer/zdcQVectors.cxx +++ b/PWGCF/Flow/TableProducer/zdcQVectors.cxx @@ -334,7 +334,6 @@ struct ZdcQVectors { } registry.fill(HIST("hEventCount"), evSel_kNoSameBunchPileup); } - if (cfgEvSelsIsGoodZvtxFT0vsPV) { if (!collision.selection_bit(o2::aod::evsel::kIsGoodZvtxFT0vsPV)) { // removes collisions with large differences between z of PV by tracks and z of PV from FT0 A-C time difference From 5bee1de306eb3cf9fe363f73e7062f3a25a59bf8 Mon Sep 17 00:00:00 2001 From: cnkoster Date: Fri, 25 Jul 2025 16:13:44 +0200 Subject: [PATCH 3/6] Add options for 2d qa track histos for centrality dependence --- PWGCF/Flow/Tasks/flowSP.cxx | 120 ++++++++++++++++++++++++++++-------- 1 file changed, 95 insertions(+), 25 deletions(-) diff --git a/PWGCF/Flow/Tasks/flowSP.cxx b/PWGCF/Flow/Tasks/flowSP.cxx index f37c15eac01..1156a8f4f43 100644 --- a/PWGCF/Flow/Tasks/flowSP.cxx +++ b/PWGCF/Flow/Tasks/flowSP.cxx @@ -14,9 +14,6 @@ /// \since 01/12/2024 /// \brief task to evaluate flow with respect to spectator plane. -#include -#include -#include #include #include #include @@ -29,6 +26,10 @@ #include "Framework/RunningWorkflowInfo.h" #include "Framework/HistogramRegistry.h" #include "Framework/O2DatabasePDGPlugin.h" +#include "CCDB/BasicCCDBManager.h" +#include "DataFormatsParameters/GRPObject.h" +#include "DataFormatsParameters/GRPMagField.h" +#include "DataFormatsParameters/GRPLHCIFData.h" #include "Common/DataModel/EventSelection.h" #include "Common/Core/TrackSelection.h" @@ -93,6 +94,7 @@ struct FlowSP { O2_DEFINE_CONFIGURABLE(cfgTrackSelsDCApt1, float, 0.1, "DcaZ < a * b / pt^1.1 -> this sets a"); O2_DEFINE_CONFIGURABLE(cfgTrackSelsDCApt2, float, 0.035, "DcaZ < a * b / pt^1.1 -> this sets b"); O2_DEFINE_CONFIGURABLE(cfgTrackSelsPIDNsigma, float, 2.0, "nSigma cut for PID"); + O2_DEFINE_CONFIGURABLE(cfgTrackSelDoTrackQAvsCent, bool, true, "Do track selection QA plots as function of centrality"); // Additional event selections O2_DEFINE_CONFIGURABLE(cfgEvSelsUseAdditionalEventCut, bool, true, "Bool to enable Additional Event Cut"); O2_DEFINE_CONFIGURABLE(cfgEvSelsMaxOccupancy, int, 10000, "Maximum occupancy of selected events"); @@ -366,15 +368,6 @@ struct FlowSP { if (cfgFillTrackQA) { registry.add("QA/after/pt_phi", "", {HistType::kTH2D, {axisPt, axisPhiMod}}); - registry.add("incl/QA/after/hPt", "", kTH1D, {axisPt}); - registry.add("incl/QA/after/hPt_forward", "", kTH1D, {axisPt}); - registry.add("incl/QA/after/hPt_forward_uncorrected", "", kTH1D, {axisPt}); - registry.add("incl/QA/after/hPt_backward", "", kTH1D, {axisPt}); - registry.add("incl/QA/after/hPt_backward_uncorrected", "", kTH1D, {axisPt}); - registry.add("incl/QA/after/hPhi", "", kTH1D, {axisPhi}); - registry.add("incl/QA/after/hPhi_uncorrected", "", kTH1D, {axisPhi}); - registry.add("incl/QA/after/hEta", "", kTH1D, {axisEta}); - registry.add("incl/QA/after/hEta_uncorrected", "", kTH1D, {axisEta}); registry.add("incl/QA/after/hPhi_Eta_vz", "", kTH3D, {axisPhi, axisEta, axisVz}); registry.add("incl/QA/after/hPhi_Eta_vz_corrected", "", kTH3D, {axisPhi, axisEta, axisVz}); registry.add("incl/QA/after/hDCAxy_pt", "", kTH2D, {axisPt, axisDCAxy}); @@ -383,6 +376,28 @@ struct FlowSP { registry.add("incl/QA/after/hCrossedRows_pt", "", {HistType::kTH2D, {axisPt, axisCl}}); registry.add("incl/QA/after/hCrossedRows_vs_SharedClusters", "", {HistType::kTH2D, {axisCl, axisShCl}}); + if(cfgTrackSelDoTrackQAvsCent){ + registry.add("incl/QA/after/hPt", "", kTH2D, {axisPt, axisCent}); + registry.add("incl/QA/after/hPt_forward", "", kTH2D, {axisPt, axisCent}); + registry.add("incl/QA/after/hPt_forward_uncorrected", "", kTH2D, {axisPt, axisCent}); + registry.add("incl/QA/after/hPt_backward", "", kTH2D, {axisPt, axisCent}); + registry.add("incl/QA/after/hPt_backward_uncorrected", "", kTH2D, {axisPt, axisCent}); + registry.add("incl/QA/after/hPhi", "", kTH2D, {axisPhi, axisCent}); + registry.add("incl/QA/after/hPhi_uncorrected", "", kTH2D, {axisPhi, axisCent}); + registry.add("incl/QA/after/hEta", "", kTH2D, {axisEta, axisCent}); + registry.add("incl/QA/after/hEta_uncorrected", "", kTH2D, {axisEta, axisCent}); + } else { + registry.add("incl/QA/after/hPt", "", kTH1D, {axisPt}); + registry.add("incl/QA/after/hPt_forward", "", kTH1D, {axisPt}); + registry.add("incl/QA/after/hPt_forward_uncorrected", "", kTH1D, {axisPt}); + registry.add("incl/QA/after/hPt_backward", "", kTH1D, {axisPt}); + registry.add("incl/QA/after/hPt_backward_uncorrected", "", kTH1D, {axisPt}); + registry.add("incl/QA/after/hPhi", "", kTH1D, {axisPhi}); + registry.add("incl/QA/after/hPhi_uncorrected", "", kTH1D, {axisPhi}); + registry.add("incl/QA/after/hEta", "", kTH1D, {axisEta}); + registry.add("incl/QA/after/hEta_uncorrected", "", kTH1D, {axisEta}); + } + if (cfgFillQABefore) registry.addClone("incl/QA/after/", "incl/QA/before/"); } @@ -641,7 +656,7 @@ struct FlowSP { return -1; } - int getMagneticField(uint64_t timestamp) + int getMagneticField(uint64_t timestamp) { // TODO done only once (and not per run). Will be replaced by CCDBConfigurable static o2::parameters::GRPMagField* grpo = nullptr; @@ -656,6 +671,19 @@ struct FlowSP { return grpo->getNominalL3Field(); } + std::pair getCrossingAngleCCDB(uint64_t timestamp) + { + // TODO done only once (and not per run). Will be replaced by CCDBConfigurable + auto grpo = ccdb->getForTimeStamp("GLO/Config/GRPLHCIF", timestamp); + if (grpo == nullptr) { + LOGF(fatal, "GRP object for Crossing Angle not found for timestamp %llu", timestamp); + return {0, 0}; + } + float crossingAngle = grpo->getCrossingAngle(); + long crossingAngleTime = grpo->getCrossingAngleTime(); + return {crossingAngle, crossingAngleTime}; + } + // From Generic Framework void loadCorrections(uint64_t timestamp) { @@ -916,7 +944,7 @@ struct FlowSP { registry.fill(HIST("QA/") + HIST(Time[ft]) + HIST("/CentFT0C_vs_CentNGlobal"), collision.centFT0C(), collision.centNGlobal(), centWeight); if (cfgFillEventPlaneQA) { - if constexpr (framework::has_type_v) { + if constexpr (o2::framework::has_type_v) { double psiA = 1.0 * std::atan2(collision.qyA(), collision.qxA()); double psiC = 1.0 * std::atan2(collision.qyC(), collision.qxC()); double psiFull = 1.0 * std::atan2(collision.qyA() + collision.qyC(), collision.qxA() + collision.qxC()); @@ -1074,6 +1102,36 @@ struct FlowSP { registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hCrossedRows_vs_SharedClusters"), track.tpcNClsFound(), track.tpcFractionSharedCls(), wacc * weff); } + template + inline void fillTrackQA(TrackObject track, double vz, double centrality, float wacc = 1, float weff = 1) + { + if (!cfgFillTrackQA) + return; + + static constexpr std::string_view Time[] = {"before/", "after/"}; + // NOTE: species[kUnidentified] = "" (when no PID) + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPt"), track.pt(), centrality, wacc * weff); + if (track.eta() > 0) { + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPt_forward"), track.pt(), centrality, wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPt_forward_uncorrected"), track.pt(), centrality); + } else { + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPt_backward"), track.pt(), centrality, wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPt_backward_uncorrected"), track.pt(), centrality); + } + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPhi"), track.phi(), centrality, wacc); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPhi_uncorrected"), track.phi(), centrality); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hEta"), track.eta(), centrality, wacc); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hEta_uncorrected"), track.eta(), centrality); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPhi_Eta_vz"), track.phi(), track.eta(), vz); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hPhi_Eta_vz_corrected"), track.phi(), track.eta(), vz, wacc); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hDCAxy_pt"), track.pt(), track.dcaXY(), wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hDCAz_pt"), track.pt(), track.dcaZ(), wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hSharedClusters_pt"), track.pt(), track.tpcFractionSharedCls(), wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hCrossedRows_pt"), track.pt(), track.tpcNClsFound(), wacc * weff); + registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hCrossedRows_vs_SharedClusters"), track.tpcNClsFound(), track.tpcFractionSharedCls(), wacc * weff); + } + + template inline void fillPIDQA(TrackObject track) { @@ -1141,15 +1199,27 @@ struct FlowSP { } template - void fillAllQA(TrackObject track, double vtxz, bool pos, float wacc = 1, float weff = 1, float waccP = 1, float weffP = 1, float waccN = 1, float weffN = 1) + void fillAllQA(TrackObject track, double vtxz, double centrality, bool pos, float wacc = 1, float weff = 1, float waccP = 1, float weffP = 1, float waccN = 1, float weffN = 1) { - fillTrackQA(track, vtxz, wacc, weff); + if(!cfgTrackSelDoTrackQAvsCent) { + fillTrackQA(track, vtxz, wacc, weff); + } else { + fillTrackQA(track, vtxz, centrality, wacc, weff); + } fillPIDQA(track); if (pos) { + if(!cfgTrackSelDoTrackQAvsCent) { fillTrackQA(track, vtxz, waccP, weffP); + } else { + fillTrackQA(track, vtxz, centrality, waccP, weffP); + } fillPIDQA(track); } else { - fillTrackQA(track, vtxz, waccN, weffN); + if(!cfgTrackSelDoTrackQAvsCent) { + fillTrackQA(track, vtxz, waccN, weffN); + } else { + fillTrackQA(track, vtxz, centrality, waccN, weffN); + } fillPIDQA(track); } } @@ -1291,16 +1361,16 @@ struct FlowSP { if (cfgFillQABefore) { switch (trackPID) { case kUnidentified: - fillAllQA(track, vtxz, pos); + fillAllQA(track, vtxz, centrality, pos); break; case kPion: - fillAllQA(track, vtxz, pos); + fillAllQA(track, vtxz, centrality, pos); break; case kKaon: - fillAllQA(track, vtxz, pos); + fillAllQA(track, vtxz, centrality, pos); break; case kProton: - fillAllQA(track, vtxz, pos); + fillAllQA(track, vtxz, centrality, pos); break; } } @@ -1345,16 +1415,16 @@ struct FlowSP { switch (trackPID) { case kUnidentified: - fillAllQA(track, vtxz, pos, wacc, weff, waccP, weffP, waccN, weffN); + fillAllQA(track, vtxz, centrality, pos, wacc, weff, waccP, weffP, waccN, weffN); break; case kPion: - fillAllQA(track, vtxz, pos, wacc, weff, waccP, weffP, waccN, weffN); + fillAllQA(track, vtxz, centrality, pos, wacc, weff, waccP, weffP, waccN, weffN); break; case kKaon: - fillAllQA(track, vtxz, pos, wacc, weff, waccP, weffP, waccN, weffN); + fillAllQA(track, vtxz, centrality, pos, wacc, weff, waccP, weffP, waccN, weffN); break; case kProton: - fillAllQA(track, vtxz, pos, wacc, weff, waccP, weffP, waccN, weffN); + fillAllQA(track, vtxz, centrality, pos, wacc, weff, waccP, weffP, waccN, weffN); break; } From 381c15e43ec27cfd8e76440ac4783c9e8e016845 Mon Sep 17 00:00:00 2001 From: ALICE Action Bot Date: Fri, 25 Jul 2025 14:14:24 +0000 Subject: [PATCH 4/6] Please consider the following formatting changes --- PWGCF/Flow/Tasks/flowSP.cxx | 85 +++++++++++++++++++------------------ 1 file changed, 43 insertions(+), 42 deletions(-) diff --git a/PWGCF/Flow/Tasks/flowSP.cxx b/PWGCF/Flow/Tasks/flowSP.cxx index 1156a8f4f43..a50fa73859e 100644 --- a/PWGCF/Flow/Tasks/flowSP.cxx +++ b/PWGCF/Flow/Tasks/flowSP.cxx @@ -14,36 +14,38 @@ /// \since 01/12/2024 /// \brief task to evaluate flow with respect to spectator plane. -#include -#include -#include -#include -#include +#include "GFWWeights.h" -#include "Framework/runDataProcessing.h" -#include "Framework/AnalysisTask.h" -#include "Framework/ASoAHelpers.h" -#include "Framework/RunningWorkflowInfo.h" -#include "Framework/HistogramRegistry.h" -#include "Framework/O2DatabasePDGPlugin.h" -#include "CCDB/BasicCCDBManager.h" -#include "DataFormatsParameters/GRPObject.h" -#include "DataFormatsParameters/GRPMagField.h" -#include "DataFormatsParameters/GRPLHCIFData.h" +#include "PWGCF/DataModel/SPTableZDC.h" -#include "Common/DataModel/EventSelection.h" +#include "Common/Core/RecoDecay.h" #include "Common/Core/TrackSelection.h" -#include "Common/DataModel/TrackSelectionTables.h" -#include "Common/DataModel/Multiplicity.h" #include "Common/DataModel/Centrality.h" -#include "Common/Core/RecoDecay.h" +#include "Common/DataModel/EventSelection.h" +#include "Common/DataModel/Multiplicity.h" #include "Common/DataModel/PIDResponse.h" +#include "Common/DataModel/TrackSelectionTables.h" + +#include "CCDB/BasicCCDBManager.h" +#include "DataFormatsParameters/GRPLHCIFData.h" +#include "DataFormatsParameters/GRPMagField.h" +#include "DataFormatsParameters/GRPObject.h" +#include "Framework/ASoAHelpers.h" +#include "Framework/AnalysisTask.h" +#include "Framework/HistogramRegistry.h" +#include "Framework/O2DatabasePDGPlugin.h" +#include "Framework/RunningWorkflowInfo.h" +#include "Framework/runDataProcessing.h" -#include "PWGCF/DataModel/SPTableZDC.h" -#include "GFWWeights.h" #include "TF1.h" #include "TPDGCode.h" +#include +#include +#include +#include +#include + using namespace o2; using namespace o2::framework; using namespace o2::framework::expressions; @@ -376,7 +378,7 @@ struct FlowSP { registry.add("incl/QA/after/hCrossedRows_pt", "", {HistType::kTH2D, {axisPt, axisCl}}); registry.add("incl/QA/after/hCrossedRows_vs_SharedClusters", "", {HistType::kTH2D, {axisCl, axisShCl}}); - if(cfgTrackSelDoTrackQAvsCent){ + if (cfgTrackSelDoTrackQAvsCent) { registry.add("incl/QA/after/hPt", "", kTH2D, {axisPt, axisCent}); registry.add("incl/QA/after/hPt_forward", "", kTH2D, {axisPt, axisCent}); registry.add("incl/QA/after/hPt_forward_uncorrected", "", kTH2D, {axisPt, axisCent}); @@ -386,8 +388,8 @@ struct FlowSP { registry.add("incl/QA/after/hPhi_uncorrected", "", kTH2D, {axisPhi, axisCent}); registry.add("incl/QA/after/hEta", "", kTH2D, {axisEta, axisCent}); registry.add("incl/QA/after/hEta_uncorrected", "", kTH2D, {axisEta, axisCent}); - } else { - registry.add("incl/QA/after/hPt", "", kTH1D, {axisPt}); + } else { + registry.add("incl/QA/after/hPt", "", kTH1D, {axisPt}); registry.add("incl/QA/after/hPt_forward", "", kTH1D, {axisPt}); registry.add("incl/QA/after/hPt_forward_uncorrected", "", kTH1D, {axisPt}); registry.add("incl/QA/after/hPt_backward", "", kTH1D, {axisPt}); @@ -656,7 +658,7 @@ struct FlowSP { return -1; } - int getMagneticField(uint64_t timestamp) + int getMagneticField(uint64_t timestamp) { // TODO done only once (and not per run). Will be replaced by CCDBConfigurable static o2::parameters::GRPMagField* grpo = nullptr; @@ -674,14 +676,14 @@ struct FlowSP { std::pair getCrossingAngleCCDB(uint64_t timestamp) { // TODO done only once (and not per run). Will be replaced by CCDBConfigurable - auto grpo = ccdb->getForTimeStamp("GLO/Config/GRPLHCIF", timestamp); - if (grpo == nullptr) { - LOGF(fatal, "GRP object for Crossing Angle not found for timestamp %llu", timestamp); - return {0, 0}; - } - float crossingAngle = grpo->getCrossingAngle(); - long crossingAngleTime = grpo->getCrossingAngleTime(); - return {crossingAngle, crossingAngleTime}; + auto grpo = ccdb->getForTimeStamp("GLO/Config/GRPLHCIF", timestamp); + if (grpo == nullptr) { + LOGF(fatal, "GRP object for Crossing Angle not found for timestamp %llu", timestamp); + return {0, 0}; + } + float crossingAngle = grpo->getCrossingAngle(); + long crossingAngleTime = grpo->getCrossingAngleTime(); + return {crossingAngle, crossingAngleTime}; } // From Generic Framework @@ -1131,7 +1133,6 @@ struct FlowSP { registry.fill(HIST(Charge[ct]) + HIST(Species[pt]) + HIST("QA/") + HIST(Time[ft]) + HIST("hCrossedRows_vs_SharedClusters"), track.tpcNClsFound(), track.tpcFractionSharedCls(), wacc * weff); } - template inline void fillPIDQA(TrackObject track) { @@ -1201,21 +1202,21 @@ struct FlowSP { template void fillAllQA(TrackObject track, double vtxz, double centrality, bool pos, float wacc = 1, float weff = 1, float waccP = 1, float weffP = 1, float waccN = 1, float weffN = 1) { - if(!cfgTrackSelDoTrackQAvsCent) { + if (!cfgTrackSelDoTrackQAvsCent) { fillTrackQA(track, vtxz, wacc, weff); } else { fillTrackQA(track, vtxz, centrality, wacc, weff); - } + } fillPIDQA(track); if (pos) { - if(!cfgTrackSelDoTrackQAvsCent) { - fillTrackQA(track, vtxz, waccP, weffP); - } else { - fillTrackQA(track, vtxz, centrality, waccP, weffP); - } + if (!cfgTrackSelDoTrackQAvsCent) { + fillTrackQA(track, vtxz, waccP, weffP); + } else { + fillTrackQA(track, vtxz, centrality, waccP, weffP); + } fillPIDQA(track); } else { - if(!cfgTrackSelDoTrackQAvsCent) { + if (!cfgTrackSelDoTrackQAvsCent) { fillTrackQA(track, vtxz, waccN, weffN); } else { fillTrackQA(track, vtxz, centrality, waccN, weffN); From db4ded5d9321b749573b5cf01d2cb0d7ab64efb4 Mon Sep 17 00:00:00 2001 From: cnkoster Date: Fri, 25 Jul 2025 16:24:41 +0200 Subject: [PATCH 5/6] Fix MegaLinter errors --- PWGCF/Flow/Tasks/flowSP.cxx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/PWGCF/Flow/Tasks/flowSP.cxx b/PWGCF/Flow/Tasks/flowSP.cxx index a50fa73859e..9c7d1124f3d 100644 --- a/PWGCF/Flow/Tasks/flowSP.cxx +++ b/PWGCF/Flow/Tasks/flowSP.cxx @@ -45,6 +45,7 @@ #include #include #include + #include using namespace o2; using namespace o2::framework; @@ -673,7 +674,7 @@ struct FlowSP { return grpo->getNominalL3Field(); } - std::pair getCrossingAngleCCDB(uint64_t timestamp) + std::pair getCrossingAngleCCDB(uint64_t timestamp) { // TODO done only once (and not per run). Will be replaced by CCDBConfigurable auto grpo = ccdb->getForTimeStamp("GLO/Config/GRPLHCIF", timestamp); @@ -682,7 +683,7 @@ struct FlowSP { return {0, 0}; } float crossingAngle = grpo->getCrossingAngle(); - long crossingAngleTime = grpo->getCrossingAngleTime(); + uint16_t crossingAngleTime = grpo->getCrossingAngleTime(); return {crossingAngle, crossingAngleTime}; } From 6c66be8d89e9307f4d722197f8d3bebe45705479 Mon Sep 17 00:00:00 2001 From: ALICE Action Bot Date: Fri, 25 Jul 2025 14:25:18 +0000 Subject: [PATCH 6/6] Please consider the following formatting changes --- PWGCF/Flow/Tasks/flowSP.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PWGCF/Flow/Tasks/flowSP.cxx b/PWGCF/Flow/Tasks/flowSP.cxx index 9c7d1124f3d..1442509bba0 100644 --- a/PWGCF/Flow/Tasks/flowSP.cxx +++ b/PWGCF/Flow/Tasks/flowSP.cxx @@ -44,8 +44,8 @@ #include #include #include +#include #include - #include using namespace o2; using namespace o2::framework;