From ad3ae407ce0cc9170cde9da4e8f8e39e556cd59d Mon Sep 17 00:00:00 2001 From: baasingh Date: Fri, 23 Jan 2026 14:26:57 +0530 Subject: [PATCH 1/3] PWGCF: fix bug --- PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx | 4 ---- 1 file changed, 4 deletions(-) diff --git a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx index 6e39fc150ff..dd074339715 100644 --- a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx +++ b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx @@ -532,8 +532,6 @@ struct FactorialMomentsTask { for (int iPt = 0; iPt < numPt; ++iPt) { if (countTracks[iPt] > 0) { mHistArrQA[iPt * 4 + 3]->Fill(countTracks[iPt]); - } else { - return; } } @@ -589,8 +587,6 @@ struct FactorialMomentsTask { for (int iPt = 0; iPt < numPt; ++iPt) { if (countTracks[iPt] > 0) { mHistArrQA[iPt * 4 + 3]->Fill(countTracks[iPt]); - } else { - return; } } // Calculate the normalized factorial moments From 03c3531dde501449e844110ac79ffff9b985f0e5 Mon Sep 17 00:00:00 2001 From: baasingh Date: Sun, 15 Feb 2026 15:40:49 +0530 Subject: [PATCH 2/3] PWGCF:FIX RUN3-MC PROCESS FUNCTION_INCLUDE SOME CONFIGURABLES Signed-off-by: baasingh --- .../EbyEFluctuations/Tasks/FactorialMomentsTask.cxx | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx index dd074339715..f91bfb86088 100644 --- a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx +++ b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx @@ -41,6 +41,9 @@ using namespace o2::framework::expressions; TH1D* tmpFqErr[6][5][52]; struct FactorialMomentsTask { + Configurable useITS{"useITS", false, "Select tracks with ITS"}; + Configurable useTPC{"useTPC", false, "Select tracks with TPC"}; + Configurable useGlobal{"useGlobal", true, "Select global tracks"}; Configurable applyCheckPtForRec{"applyCheckPtForRec", false, "Apply checkpT for reconstructed tracks"}; Configurable applyCheckPtForMC{"applyCheckPtForMC", true, "Apply checkpT for MC-generated tracks"}; Configurable centralEta{"centralEta", 0.9, "eta limit for tracks"}; @@ -402,9 +405,12 @@ struct FactorialMomentsTask { fqEvent = {{{{{0, 0, 0, 0, 0, 0}}}}}; binConEvent = {{{0, 0, 0, 0, 0}}}; for (auto const& track : colltracks) { - // if (track.hasITS()) - // if (track.hasTPC()) - // if (track.isGlobalTrack()) { + if (useITS && !track.hasITS()) + continue; + if (useTPC && !track.hasTPC()) + continue; + if (useGlobal && !track.isGlobalTrack()) + continue; histos.fill(HIST("mCollID"), track.collisionId()); histos.fill(HIST("mEta"), track.eta()); histos.fill(HIST("mPt"), track.pt()); From eb80df8c307f09f1b6bbca875d5188235ec2d64b Mon Sep 17 00:00:00 2001 From: baasingh Date: Fri, 20 Feb 2026 16:13:01 +0530 Subject: [PATCH 3/3] PWGCF/EbyE:MINOR FIX, Need to insert some missing condition in order to tune the process function with configurables Signed-off-by: baasingh --- PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx index f91bfb86088..9ca0bbb7e3f 100644 --- a/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx +++ b/PWGCF/EbyEFluctuations/Tasks/FactorialMomentsTask.cxx @@ -431,8 +431,9 @@ struct FactorialMomentsTask { histos.fill(HIST("mNFractionShClsTPC"), track.tpcFractionSharedCls()); histos.fill(HIST("mSharedClsvsPt"), track.pt(), track.tpcNClsShared()); histos.fill(HIST("mSharedClsProbvsPt"), track.pt(), track.tpcFractionSharedCls() / track.tpcNClsCrossedRows()); - checkpT(track); - //} + if (applyCheckPtForRec && !applyCheckPtForMC) { + checkpT(track); + } } auto mcParts = mcParticles.sliceBy(perMcCollision, coll.mcCollision().globalIndex()); for (auto const& mc : mcParts) {