Skip to content

Commit ca1a6c7

Browse files
authored
[PWGCF] fix labeled array configurable in twoParticleCorrelationsMPI.cxx (#17164)
1 parent d1123d5 commit ca1a6c7

3 files changed

Lines changed: 9 additions & 23 deletions

File tree

PWGCF/GenericFramework/Tasks/flowGenericFramework.cxx

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@ struct FlowGenericFramework {
715715
registryQA.add("K0/PiMinusTOF_K0", "", {HistType::kTH2D, {{ptAxis, axisNsigmaTOF}}});
716716
registryQA.add("K0/hK0Phi", "", {HistType::kTH1D, {phiAxis}});
717717
registryQA.add("K0/hK0Eta", "", {HistType::kTH1D, {etaAxis}});
718-
registryQA.add("K0/hK0AP", "", {HistType::kTH2D, {{100, -1, 1}, {100, 0, 5.}}});
718+
registryQA.add("K0/hK0AP", "", {HistType::kTH2D, {{100, -1, 1}, {500, 0, 1.}}});
719719
registryQA.add("K0/hK0Mass_sparse", "", {HistType::kTHnSparseF, {{axisK0Mass, ptAxis, nchAxis}}});
720720
registryQA.add("K0/hK0s", "", {HistType::kTH1D, {singleCount}});
721721
registryQA.add("K0/hK0s_corrected", "", {HistType::kTH1D, {singleCount}});
@@ -2916,20 +2916,6 @@ struct FlowGenericFramework {
29162916
return track.eta() > cfgKinematics.cfgEta->first && track.eta() < cfgKinematics.cfgEta->second;
29172917
}
29182918

2919-
template <typename TParticle>
2920-
bool areGeneratedDaughtersWithinEfficiencyEtaAcceptance(const TParticle& particle)
2921-
{
2922-
if (!particle.has_daughters()) {
2923-
return false;
2924-
}
2925-
for (const auto& daughter : particle.template daughters_as<aod::McParticles>()) {
2926-
if (daughter.eta() <= cfgKinematics.cfgEta->first || daughter.eta() >= cfgKinematics.cfgEta->second) {
2927-
return false;
2928-
}
2929-
}
2930-
return true;
2931-
}
2932-
29332919
void fillV0EfficiencyRecoDebug(float pt, float centrality, int particleIndex, int stage)
29342920
{
29352921
registry.fill(HIST("Efficiency/v0RecoDebug"), pt, centrality, particleIndex, stage);
@@ -3010,7 +2996,7 @@ struct FlowGenericFramework {
30102996
return;
30112997
}
30122998
fillV0EfficiencyRecoDebug(posMother.pt(), centrality, EfficiencyK0, V0EfficiencySelection);
3013-
if (!areGeneratedDaughtersWithinEfficiencyEtaAcceptance(posMother)) {
2999+
if (!isWithinEfficiencyEtaAcceptance(posMcParticle) || !isWithinEfficiencyEtaAcceptance(negMcParticle)) {
30143000
return;
30153001
}
30163002
fillV0EfficiencyRecoDebug(posMother.pt(), centrality, EfficiencyK0, V0EfficiencyGeneratedDaughterEta);
@@ -3035,7 +3021,7 @@ struct FlowGenericFramework {
30353021
return;
30363022
}
30373023
fillV0EfficiencyRecoDebug(posMother.pt(), centrality, EfficiencyLambda, V0EfficiencySelection);
3038-
if (!areGeneratedDaughtersWithinEfficiencyEtaAcceptance(posMother)) {
3024+
if (!isWithinEfficiencyEtaAcceptance(posMcParticle) || !isWithinEfficiencyEtaAcceptance(negMcParticle)) {
30393025
return;
30403026
}
30413027
fillV0EfficiencyRecoDebug(posMother.pt(), centrality, EfficiencyLambda, V0EfficiencyGeneratedDaughterEta);

PWGCF/GenericFramework/Tasks/flowGfwNonflow.cxx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -243,11 +243,11 @@ struct FlowGfwNonflow {
243243

244244
// Generic Framework
245245
GFW* fGFW = new GFW();
246-
std::vector<GFW::CorrConfig> corrconfigs{};
246+
std::vector<GFW::CorrConfig> corrconfigs;
247247
TRandom3* fRndm = new TRandom3(0);
248248
TAxis* fPtAxis = nullptr;
249249
int lastRun = -1;
250-
std::vector<std::string> multipletKeys{};
250+
std::vector<std::string> multipletKeys;
251251

252252
// Track selection - DCA functions
253253
TF1* fPtDepDCAxy = nullptr;
@@ -420,7 +420,7 @@ struct FlowGfwNonflow {
420420
delete oba;
421421

422422
// Identify unique multiplet setups and populate with identifying keys
423-
identifyUniqueMultipletKeys(multipletKeys, corrconfigs, centAxis); // For now keep centrality axis hardcoded
423+
identifyUniqueMultipletKeys(multipletKeys, corrconfigs, multAxis);
424424
LOGF(info, "Unique multiplet keys for current configuration setup");
425425
for (const auto& key : multipletKeys) {
426426
LOGF(info, key);
@@ -524,7 +524,7 @@ struct FlowGfwNonflow {
524524
}
525525
}
526526

527-
void identifyUniqueMultipletKeys(std::vector<std::string>& keys, const std::vector<GFW::CorrConfig>& configs, AxisSpec xAxis)
527+
void identifyUniqueMultipletKeys(std::vector<std::string>& keys, const std::vector<GFW::CorrConfig>& configs, const AxisSpec& xAxis)
528528
{
529529
std::set<std::string> uniqueKeys(keys.begin(), keys.end());
530530
const auto& regionNames = gfwMemberCache.regions.GetNames();
@@ -584,7 +584,7 @@ struct FlowGfwNonflow {
584584

585585
if (uniqueKeys.insert(key).second) {
586586
keys.push_back(std::move(key));
587-
MultipletConfig currentMultipletConfig{configIndex, totalOrder, registry.add<TProfile>(Form("%s", keys.back().c_str()), Form("; centrality ; N_{%d}", totalOrder), {HistType::kTProfile, {xAxis}})};
587+
MultipletConfig currentMultipletConfig{.representativeConfigIndex = configIndex, .order = totalOrder, .profile = registry.add<TProfile>(Form("%s", keys.back().c_str()), Form("; centrality ; N_{%d}", totalOrder), {HistType::kTProfile, {xAxis}})};
588588
effectiveMultiplets[keys.back()] = currentMultipletConfig;
589589
}
590590
}

PWGCF/TwoParticleCorrelations/Tasks/twoParticleCorrelationsMpi.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ struct TwoParticleCorrelationsMpi {
9999
Configurable<std::string> cfgMultCutFormula{"cfgMultCutFormula", "", "Multiplicity correlations cut formula. A result greater than zero results in accepted event. Parameters: [cFT0C] FT0C centrality, [mFV0A] V0A multiplicity, [mGlob] global track multiplicity, [mPV] PV track multiplicity, [cFT0M] FT0M centrality"};
100100

101101
// Suggested values: Photon: 0.004; K0 and Lambda: 0.005
102-
Configurable<LabeledArray<float>> cfgPairCut{"cfgPairCut", {CfgPairCutDefaults.front().data(), 5, 1, {"Photon", "K0", "Lambda", "Phi", "Rho"}}, "Pair cuts on various particles"};
102+
Configurable<LabeledArray<float>> cfgPairCut{"cfgPairCut", {CfgPairCutDefaults.front().data(), 5, {"Photon", "K0", "Lambda", "Phi", "Rho"}}, "Pair cuts on various particles"};
103103

104104
Configurable<std::string> cfgEfficiencyTrigger{"cfgEfficiencyTrigger", "", "CCDB path to efficiency object for trigger particles"};
105105
Configurable<std::string> cfgEfficiencyAssociated{"cfgEfficiencyAssociated", "", "CCDB path to efficiency object for associated particles"};

0 commit comments

Comments
 (0)