From 9d057a6ef5aa8b962a22faa75322a89c8196461c Mon Sep 17 00:00:00 2001 From: ALICE Action Bot Date: Sun, 24 Nov 2024 22:32:18 +0000 Subject: [PATCH] Please consider the following formatting changes --- .../TableProducer/tableMakerMC_withAssoc.cxx | 11 ++- PWGDQ/TableProducer/tableMaker_withAssoc.cxx | 7 +- PWGDQ/Tasks/dqEfficiency_withAssoc.cxx | 70 +++++++++---------- PWGDQ/Tasks/tableReader_withAssoc.cxx | 39 +++++------ 4 files changed, 61 insertions(+), 66 deletions(-) diff --git a/PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx b/PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx index ef96b4ec5ef..241cd5d74a2 100644 --- a/PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx +++ b/PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx @@ -218,7 +218,7 @@ struct TableMakerMC { if (!(isProcessBCenabled || isBarrelEnabled || isMuonEnabled)) { LOG(fatal) << "No process function was enabled for TableMakerMC. Check it out!!!"; } - + // Define user specified cut DefineCuts(); @@ -368,7 +368,6 @@ struct TableMakerMC { } } - void skimMCParticles(aod::McParticles const& mcTracks, aod::McCollisions const&) { // Select MC particles which fulfill at least one of the user specified MC signals @@ -381,7 +380,7 @@ struct TableMakerMC { fLabelsMapReversed.clear(); fMCFlags.clear(); - uint16_t mcflags = static_cast(0); // flags which will hold the decisions for each MC signal + uint16_t mcflags = static_cast(0); // flags which will hold the decisions for each MC signal int trackCounter = 0; for (auto& mctrack : mcTracks) { @@ -451,7 +450,7 @@ struct TableMakerMC { // Loop over collisions for (const auto& collision : collisions) { - // Fill the stats event histogram with the event selection bits + // Fill the stats event histogram with the event selection bits for (int i = 0; i < o2::aod::evsel::kNsel; i++) { if (collision.selection_bit(i)) { (reinterpret_cast(fStatsList->At(0)))->Fill(1.0, static_cast(i)); @@ -591,7 +590,7 @@ struct TableMakerMC { continue; } - // If this track is already present in the index map, it means it was already skimmed, + // If this track is already present in the index map, it means it was already skimmed, // so we just store the association and we skip the track if (fTrackIndexMap.find(track.globalIndex()) != fTrackIndexMap.end()) { trackBarrelAssoc(fCollIndexMap[collision.globalIndex()], fTrackIndexMap[track.globalIndex()]); @@ -616,7 +615,7 @@ struct TableMakerMC { // However, in data analysis one should loop over associations, so this one should not be used. // In the case of Run2-like analysis, there will be no associations, so this ID will be the one originally assigned in the AO2Ds (updated for the skims) uint32_t reducedEventIdx = fCollIndexMap[track.collisionId()]; - + // NOTE: trackBarrelInfo stores the index of the collision as in AO2D (for use in some cases where the analysis on skims is done // in workflows where the original AO2Ds are also present) trackBarrelInfo(track.collisionId(), collision.posX(), collision.posY(), collision.posZ(), track.globalIndex()); diff --git a/PWGDQ/TableProducer/tableMaker_withAssoc.cxx b/PWGDQ/TableProducer/tableMaker_withAssoc.cxx index cb476da72dd..8f437b7955c 100644 --- a/PWGDQ/TableProducer/tableMaker_withAssoc.cxx +++ b/PWGDQ/TableProducer/tableMaker_withAssoc.cxx @@ -497,7 +497,6 @@ struct TableMaker { fStatsList->AddAt(histZorroSel, kStatsZorroSel); } - template void skimCollisions(TEvents const& collisions, TBCs const& /*bcs*/, TZdcs const& /*zdcs*/, TTrackAssoc const& trackAssocs, TTracks const& tracks) @@ -703,7 +702,7 @@ struct TableMaker { continue; } - // If this track is already present in the index map, it means it was already skimmed, + // If this track is already present in the index map, it means it was already skimmed, // so we just store the association and we skip the track if (fTrackIndexMap.find(track.globalIndex()) != fTrackIndexMap.end()) { trackBarrelAssoc(fCollIndexMap[collision.globalIndex()], fTrackIndexMap[track.globalIndex()]); @@ -761,7 +760,7 @@ struct TableMaker { // If new associations are done with the skimmed data, the track time wrt new collision can then be recomputed based on the // relative difference in time between the original and the new collision. uint32_t reducedEventIdx = fCollIndexMap[track.collisionId()]; // This gives the original iD of the track - + // NOTE: trackBarrelInfo stores the index of the collision as in AO2D (for use in some cases where the analysis on skims is done // in workflows where the original AO2Ds are also present) trackBarrelInfo(collision.globalIndex(), collision.posX(), collision.posY(), collision.posZ(), track.globalIndex()); @@ -790,7 +789,7 @@ struct TableMaker { track.trdSignal()); } fTrackIndexMap[track.globalIndex()] = trackBasic.lastIndex(); - + // write the skimmed collision - track association trackBarrelAssoc(fCollIndexMap[collision.globalIndex()], fTrackIndexMap[track.globalIndex()]); } // end loop over associations diff --git a/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx b/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx index 3466b2192bb..f34c5d44d10 100644 --- a/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx +++ b/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx @@ -206,7 +206,6 @@ struct AnalysisEventSelection { fSelMap.clear(); fBCCollMap.clear(); - for (auto& event : events) { // Reset the fValues array and fill event observables VarManager::ResetValues(0, VarManager::kNEventWiseVariables); @@ -255,7 +254,7 @@ struct AnalysisEventSelection { for (auto bc1It = fBCCollMap.begin(); bc1It != fBCCollMap.end(); ++bc1It) { uint64_t bc1 = bc1It->first; auto bc1Events = bc1It->second; - + // same bunch event correlations, if more than 1 collisions in this bunch if (bc1Events.size() > 1) { for (auto ev1It = bc1Events.begin(); ev1It != bc1Events.end(); ++ev1It) { @@ -269,9 +268,9 @@ struct AnalysisEventSelection { collisionSplittingMap[*ev2It] = true; } fHistMan->FillHistClass("SameBunchCorrelations", VarManager::fgValues); - } // end second event loop - } // end first event loop - } // end if BC1 events > 1 + } // end second event loop + } // end first event loop + } // end if BC1 events > 1 // loop over the following BCs in the TF for (auto bc2It = std::next(bc1It); bc2It != fBCCollMap.end(); ++bc2It) { @@ -537,8 +536,8 @@ struct AnalysisTrackSelection { } } } // end loop over cuts - } // end loop over MC signals - } // end if (filterMap > 0) + } // end loop over MC signals + } // end if (filterMap > 0) // count the number of associations per track if (fConfigPublishAmbiguity && filterMap > 0) { @@ -818,10 +817,10 @@ struct AnalysisMuonSelection { } } } // end loop over cuts - } // end loop over MC signals + } // end loop over MC signals // count the number of associations per track - if (fConfigPublishAmbiguity && filterMap > 0) { + if (fConfigPublishAmbiguity && filterMap > 0) { if (event.isEventSelected_bit(1)) { if (fNAssocsInBunch.find(track.globalIndex()) == fNAssocsInBunch.end()) { std::vector evVector = {event.globalIndex()}; @@ -829,7 +828,7 @@ struct AnalysisMuonSelection { } else { auto& evVector = fNAssocsInBunch[track.globalIndex()]; evVector.push_back(event.globalIndex()); - } + } } else { if (fNAssocsOutOfBunch.find(track.globalIndex()) == fNAssocsOutOfBunch.end()) { std::vector evVector = {event.globalIndex()}; @@ -846,9 +845,9 @@ struct AnalysisMuonSelection { // TODO: some tracks can be associated to both collisions that have in bunch pileup and collisions from different bunches // So one could QA these tracks separately if (fConfigPublishAmbiguity) { - for (auto& [trackIdx, evIndices] : fNAssocsInBunch) { + for (auto& [trackIdx, evIndices] : fNAssocsInBunch) { if (evIndices.size() == 1) { - continue; + continue; } auto track = muons.rawIteratorAt(trackIdx); VarManager::ResetValues(0, VarManager::kNVars); @@ -859,7 +858,7 @@ struct AnalysisMuonSelection { for (auto& [trackIdx, evIndices] : fNAssocsOutOfBunch) { if (evIndices.size() == 1) { - continue; + continue; } auto track = muons.rawIteratorAt(trackIdx); VarManager::ResetValues(0, VarManager::kNVars); @@ -960,12 +959,12 @@ struct AnalysisPrefilterSelection { } } } - // NOTE: If no prefilter loose cut is specified to be "prefiltered" or no track cuts are asked to be prefiltered, + // NOTE: If no prefilter loose cut is specified to be "prefiltered" or no track cuts are asked to be prefiltered, // then make a warning. In the processing, the produced table will just lead to a neutral behaviour (no prefilter applied) if (fPrefilterMask == static_cast(0) || fPrefilterCutBit < 0) { LOG(warn) << "No loose cut or track cuts for prefiltering are specified. This task will do nothing."; } - + // setup the prefilter pair cut fPairCut = new AnalysisCompositeCut(true); TString pairCutStr = fConfigPrefilterPairCut.value; @@ -1042,8 +1041,7 @@ struct AnalysisPrefilterSelection { // If cuts were not configured, then produce a map with all 1's if (fPrefilterCutBit < 0 || fPrefilterMask == 0) { prefilter(mymap); - } - else if (fPrefilterMap.find(track.globalIndex()) != fPrefilterMap.end()) { + } else if (fPrefilterMap.find(track.globalIndex()) != fPrefilterMap.end()) { // NOTE: publish the bitwise negated bits (~), so there will be zeroes for cuts that failed the prefiltering and 1 everywhere else mymap = ~fPrefilterMap[track.globalIndex()]; prefilter(mymap); @@ -1095,18 +1093,18 @@ struct AnalysisSameEventPairing { Configurable lutPath{"lutPath", "GLO/Param/MatLUT", "Path of the Lut parametrization"}; Configurable geoPath{"geoPath", "GLO/Config/GeometryAligned", "Path of the geometry file"}; } fConfigCCDB; - + struct : ConfigurableGroup { - Configurable useRemoteField{"cfgUseRemoteField", false, "Chose whether to fetch the magnetic field from ccdb or set it manually"}; - Configurable magField{"cfgMagField", 5.0f, "Manually set magnetic field"}; - Configurable flatTables{"cfgFlatTables", false, "Produce a single flat tables with all relevant information of the pairs and single tracks"}; - Configurable useKFVertexing{"cfgUseKFVertexing", false, "Use KF Particle for secondary vertex reconstruction (DCAFitter is used by default)"}; - Configurable useAbsDCA{"cfgUseAbsDCA", false, "Use absolute DCA minimization instead of chi^2 minimization in secondary vertexing"}; - Configurable propToPCA{"cfgPropToPCA", false, "Propagate tracks to secondary vertex"}; - Configurable corrFullGeo{"cfgCorrFullGeo", false, "Use full geometry to correct for MCS effects in track propagation"}; - Configurable noCorr{"cfgNoCorrFwdProp", false, "Do not correct for MCS effects in track propagation"}; - Configurable collisionSystem{"syst", "pp", "Collision system, pp or PbPb"}; - Configurable centerMassEnergy{"energy", 13600, "Center of mass energy in GeV"}; + Configurable useRemoteField{"cfgUseRemoteField", false, "Chose whether to fetch the magnetic field from ccdb or set it manually"}; + Configurable magField{"cfgMagField", 5.0f, "Manually set magnetic field"}; + Configurable flatTables{"cfgFlatTables", false, "Produce a single flat tables with all relevant information of the pairs and single tracks"}; + Configurable useKFVertexing{"cfgUseKFVertexing", false, "Use KF Particle for secondary vertex reconstruction (DCAFitter is used by default)"}; + Configurable useAbsDCA{"cfgUseAbsDCA", false, "Use absolute DCA minimization instead of chi^2 minimization in secondary vertexing"}; + Configurable propToPCA{"cfgPropToPCA", false, "Propagate tracks to secondary vertex"}; + Configurable corrFullGeo{"cfgCorrFullGeo", false, "Use full geometry to correct for MCS effects in track propagation"}; + Configurable noCorr{"cfgNoCorrFwdProp", false, "Do not correct for MCS effects in track propagation"}; + Configurable collisionSystem{"syst", "pp", "Collision system, pp or PbPb"}; + Configurable centerMassEnergy{"energy", 13600, "Center of mass energy in GeV"}; } fConfigOptions; struct : ConfigurableGroup { @@ -1246,7 +1244,7 @@ struct AnalysisSameEventPairing { // NOTE: In the numbering scheme for the map key, we use the number of barrel cuts in the barrel-track selection task fTrackHistNames[fNCutsBarrel + icut * fNPairCuts + iPairCut] = names; } // end loop (pair cuts) - } // end if (pair cuts) + } // end if (pair cuts) // assign hist directories for the MC matched pairs for each (track cut,MCsignal) combination if (!sigNamesStr.IsNull()) { @@ -1324,7 +1322,7 @@ struct AnalysisSameEventPairing { histNames += Form("%s;%s;%s;", names[0].Data(), names[1].Data(), names[2].Data()); fMuonHistNames[fNCutsMuon + icut * fNCutsMuon + iPairCut] = names; } // end loop (pair cuts) - } // end if (pair cuts) + } // end if (pair cuts) // assign hist directories for pairs matched to MC signals for each (muon cut, MCrec signal) combination if (!sigNamesStr.IsNull()) { @@ -1353,7 +1351,7 @@ struct AnalysisSameEventPairing { } } } // end loop over cuts - } // end if (muonCutsStr) + } // end if (muonCutsStr) // Add histogram classes for each specified MCsignal at the generator level // TODO: create a std::vector of hist classes to be used at Fill time, to avoid using Form in the process function @@ -1745,8 +1743,8 @@ struct AnalysisSameEventPairing { } // end loop (pair cuts) } } // end loop (cuts) - } // end loop over pairs of track associations - } // end loop over events + } // end loop over pairs of track associations + } // end loop over events } // Preslice perReducedMcEvent = aod::reducedtrackMC::reducedMCeventId; @@ -1795,8 +1793,8 @@ struct AnalysisSameEventPairing { fHistMan->FillHistClass(Form("MCTruthGenPair_%s", sig.GetName()), VarManager::fgValues); } } // end loop over MC signals - } // end loop over pairs - } // end loop over events + } // end loop over pairs + } // end loop over events } } // end runMCGen @@ -1989,7 +1987,7 @@ struct AnalysisDileptonTrack { DefineHistograms(fHistMan, fHistNamesDileptonTrack[icut], fConfigHistogramSubgroups.value.data()); // define dilepton-track histograms DefineHistograms(fHistMan, fHistNamesDileptons[icut], "barrel,vertexing"); // define dilepton histograms std::vector mcHistNames; - + for (auto& sig : fRecMCSignals) { mcHistNames.push_back(Form("DileptonTrackMCMatched_%s_%s_%s", tempStr.Data(), fConfigTrackCut.value.data(), sig.GetName())); DefineHistograms(fHistMan, mcHistNames[mcHistNames.size() - 1], fConfigHistogramSubgroups.value.data()); diff --git a/PWGDQ/Tasks/tableReader_withAssoc.cxx b/PWGDQ/Tasks/tableReader_withAssoc.cxx index c7a3845a9ea..17894f40a9a 100644 --- a/PWGDQ/Tasks/tableReader_withAssoc.cxx +++ b/PWGDQ/Tasks/tableReader_withAssoc.cxx @@ -66,7 +66,7 @@ namespace o2::aod namespace dqanalysisflags { DECLARE_SOA_COLUMN(MixingHash, mixingHash, int); //! Hash used in event mixing -DECLARE_SOA_BITMAP_COLUMN(IsEventSelected, isEventSelected, 8); //! Event decision +DECLARE_SOA_BITMAP_COLUMN(IsEventSelected, isEventSelected, 8); //! Event decision DECLARE_SOA_BITMAP_COLUMN(IsBarrelSelected, isBarrelSelected, 32); //! Barrel track decisions DECLARE_SOA_COLUMN(BarrelAmbiguityInBunch, barrelAmbiguityInBunch, int8_t); //! Barrel track in-bunch ambiguity DECLARE_SOA_COLUMN(BarrelAmbiguityOutOfBunch, barrelAmbiguityOutOfBunch, int8_t); //! Barrel track out of bunch ambiguity @@ -168,11 +168,11 @@ struct AnalysisEventSelection { Configurable fConfigITSROFrameStartBorderMargin{"cfgITSROFrameStartBorderMargin", -1, "Number of bcs at the start of ITS RO Frame border. Take from CCDB if -1"}; Configurable fConfigITSROFrameEndBorderMargin{"cfgITSROFrameEndBorderMargin", -1, "Number of bcs at the end of ITS RO Frame border. Take from CCDB if -1"}; - + Configurable fConfigSplitCollisionsDeltaZ{"cfgSplitCollisionsDeltaZ", 1.0, "maximum delta-z (cm) between two collisions to consider them as split candidates"}; Configurable fConfigSplitCollisionsDeltaBC{"cfgSplitCollisionsDeltaBC", 100, "maximum delta-BC between two collisions to consider them as split candidates; do not apply if value is negative"}; Configurable fConfigCheckSplitCollisions{"cfgCheckSplitCollisions", false, "If true, run the split collision check and fill histograms"}; - + Configurable fConfigRunZorro{"cfgRunZorro", false, "Enable event selection with zorro [WARNING: under debug, do not enable!]"}; Configurable fConfigCcdbUrl{"ccdb-url", "http://alice-ccdb.cern.ch", "url of the ccdb repository"}; Configurable fConfigNoLaterThan{"ccdb-no-later-than", std::chrono::duration_cast(std::chrono::system_clock::now().time_since_epoch()).count(), "latest acceptable timestamp of creation for the object"}; @@ -296,7 +296,7 @@ struct AnalysisEventSelection { template void publishSelections(TEvents const& events) { - // Create a map for collisions which are candidate of being split + // Create a map for collisions which are candidate of being split // key: event global index, value: whether pileup event is a possible splitting std::map collisionSplittingMap; @@ -306,7 +306,7 @@ struct AnalysisEventSelection { for (auto bc1It = fBCCollMap.begin(); bc1It != fBCCollMap.end(); ++bc1It) { uint64_t bc1 = bc1It->first; auto bc1Events = bc1It->second; - + // same bunch event correlations, if more than 1 collisions in this bunch if (bc1Events.size() > 1) { for (auto ev1It = bc1Events.begin(); ev1It != bc1Events.end(); ++ev1It) { @@ -320,9 +320,9 @@ struct AnalysisEventSelection { collisionSplittingMap[*ev2It] = true; } fHistMan->FillHistClass("SameBunchCorrelations", VarManager::fgValues); - } // end second event loop - } // end first event loop - } // end if BC1 events > 1 + } // end second event loop + } // end first event loop + } // end if BC1 events > 1 // loop over the following BCs in the TF for (auto bc2It = std::next(bc1It); bc2It != fBCCollMap.end(); ++bc2It) { @@ -409,7 +409,7 @@ struct AnalysisTrackSelection { Configurable fConfigCcdbUrl{"ccdb-url", "http://alice-ccdb.cern.ch", "url of the ccdb repository"}; Configurable fConfigCcdbPathTPC{"ccdb-path-tpc", "Users/z/zhxiong/TPCPID/PostCalib", "base path to the ccdb object"}; Configurable fConfigNoLaterThan{"ccdb-no-later-than", std::chrono::duration_cast(std::chrono::system_clock::now().time_since_epoch()).count(), "latest acceptable timestamp of creation for the object"}; - + Configurable fConfigComputeTPCpostCalib{"cfgTPCpostCalib", false, "If true, compute TPC post-calibrated n-sigmas"}; Configurable grpmagPath{"grpmagPath", "GLO/Config/GRPMagField", "CCDB path of the GRPMagField object"}; Configurable fConfigDummyRunlist{"cfgDummyRunlist", false, "If true, use dummy runlist"}; @@ -467,7 +467,7 @@ struct AnalysisTrackSelection { if (fConfigDummyRunlist) { VarManager::SetDummyRunlist(fConfigInitRunNumber); } - + fCCDB->setURL(fConfigCcdbUrl.value); fCCDB->setCaching(true); fCCDB->setLocalObjectValidityChecking(); @@ -611,7 +611,7 @@ struct AnalysisTrackSelection { } trackAmbiguities(nInBunch, nOutOfBunch); } - } // end if (fConfigPublishAmbiguity) + } // end if (fConfigPublishAmbiguity) } // end runTrackSelection() @@ -778,7 +778,7 @@ struct AnalysisMuonSelection { evVector.push_back(event.globalIndex()); } } - } // end if (fConfigPublishAmbiguity) + } // end if (fConfigPublishAmbiguity) } // end loop over assocs if (fConfigPublishAmbiguity) { @@ -894,7 +894,7 @@ struct AnalysisPrefilterSelection { if (fPrefilterMask == static_cast(0) || fPrefilterCutBit < 0) { LOG(warn) << "No specified loose cut or track cuts for prefiltering. This task will do nothing."; } - + // setup the prefilter pair cut fPairCut = new AnalysisCompositeCut(true); TString pairCutStr = fConfigPrefilterPairCut.value; @@ -906,7 +906,7 @@ struct AnalysisPrefilterSelection { VarManager::SetDefaultVarNames(); VarManager::SetupTwoProngDCAFitter(5.0f, true, 200.0f, 4.0f, 1.0e-3f, 0.9f, true); // TODO: get these parameters from Configurables - //VarManager::SetupTwoProngFwdDCAFitter(5.0f, true, 200.0f, 1.0e-3f, 0.9f, true); + // VarManager::SetupTwoProngFwdDCAFitter(5.0f, true, 200.0f, 1.0e-3f, 0.9f, true); } template @@ -971,8 +971,7 @@ struct AnalysisPrefilterSelection { // If cuts were not configured, then produce a map with all 1's if (fPrefilterCutBit < 0 || fPrefilterMask == 0) { prefilter(mymap); - } - else if (fPrefilterMap.find(track.globalIndex()) != fPrefilterMap.end()) { + } else if (fPrefilterMap.find(track.globalIndex()) != fPrefilterMap.end()) { // NOTE: publish the bitwise negated bits (~), so there will be zeroes for cuts that failed the prefiltering and 1 everywhere else mymap = ~fPrefilterMap[track.globalIndex()]; prefilter(mymap); @@ -1031,7 +1030,7 @@ struct AnalysisSameEventPairing { struct : ConfigurableGroup { Configurable useRemoteField{"cfgUseRemoteField", false, "Chose whether to fetch the magnetic field from ccdb or set it manually"}; - Configurable magField{"cfgMagField", 5.0f, "Manually set magnetic field"}; + Configurable magField{"cfgMagField", 5.0f, "Manually set magnetic field"}; Configurable flatTables{"cfgFlatTables", false, "Produce a single flat tables with all relevant information of the pairs and single tracks"}; Configurable useKFVertexing{"cfgUseKFVertexing", false, "Use KF Particle for secondary vertex reconstruction (DCAFitter is used by default)"}; Configurable useAbsDCA{"cfgUseAbsDCA", false, "Use absolute DCA minimization instead of chi^2 minimization in secondary vertexing"}; @@ -1286,8 +1285,8 @@ struct AnalysisSameEventPairing { fHistMan->SetUseDefaultVariableNames(true); fHistMan->SetDefaultVarNames(VarManager::fgVariableNames, VarManager::fgVariableUnits); VarManager::SetCollisionSystem((TString)fConfigOptions.collisionSystem, fConfigOptions.centerMassEnergy); // set collision system and center of mass energy - DefineHistograms(fHistMan, histNames.Data(), fConfigAddSEPHistogram.value.data()); // define all histograms - VarManager::SetUseVars(fHistMan->GetUsedVars()); // provide the list of required variables so that VarManager knows what to fill + DefineHistograms(fHistMan, histNames.Data(), fConfigAddSEPHistogram.value.data()); // define all histograms + VarManager::SetUseVars(fHistMan->GetUsedVars()); // provide the list of required variables so that VarManager knows what to fill fOutputList.setObject(fHistMan->GetMainHistogramList()); } } @@ -2544,7 +2543,7 @@ struct AnalysisDileptonTrack { // TODO: The filter expressions seem to always use the default value of configurables, not the values from the actual configuration file Filter eventFilter = aod::dqanalysisflags::isEventSelected > static_cast(1); - Filter dileptonFilter = aod::reducedpair::pt > fConfigDileptonpTCut && aod::reducedpair::mass > fConfigDileptonLowMass && aod::reducedpair::mass fConfigDileptonLxyCut; + Filter dileptonFilter = aod::reducedpair::pt > fConfigDileptonpTCut&& aod::reducedpair::mass > fConfigDileptonLowMass&& aod::reducedpair::mass fConfigDileptonLxyCut; Filter filterBarrel = aod::dqanalysisflags::isBarrelSelected > static_cast(0); Filter filterMuon = aod::dqanalysisflags::isMuonSelected > static_cast(0);