Skip to content

Commit e0e0b91

Browse files
authored
[PWGJE] removing circular depenency for background subtraction (AliceO2Group#11090)
1 parent 0d333c4 commit e0e0b91

File tree

3 files changed

+4
-17
lines changed

3 files changed

+4
-17
lines changed

PWGJE/TableProducer/rhoEstimator.cxx

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,6 @@ struct RhoEstimatorTask {
7070
Configurable<float> bkgPhiMax{"bkgPhiMax", 99., "maximum phi for determining background density"};
7171
Configurable<bool> doSparse{"doSparse", false, "perfom sparse estimation"};
7272

73-
Configurable<float> thresholdChargedJetPtMin{"thresholdChargedJetPtMin", 0.0, "Minimum charged jet pt to accept event"};
74-
Configurable<float> thresholdNeutralJetPtMin{"thresholdNeutralJetPtMin", 0.0, "Minimum neutral jet pt to accept event"};
75-
Configurable<float> thresholdFullJetPtMin{"thresholdFullJetPtMin", 0.0, "Minimum full jet pt to accept event"};
7673
Configurable<float> thresholdTriggerTrackPtMin{"thresholdTriggerTrackPtMin", 0.0, "Minimum trigger track pt to accept event"};
7774
Configurable<float> thresholdClusterEnergyMin{"thresholdClusterEnergyMin", 0.0, "Minimum cluster energy to accept event"};
7875
Configurable<bool> performTriggerTrackSelection{"performTriggerTrackSelection", false, "only accept trigger tracks that pass one of the track selections"};
@@ -141,13 +138,7 @@ struct RhoEstimatorTask {
141138
void processSelectionObjects(T& selectionObjects)
142139
{
143140
float selectionObjectPtMin = 0.0;
144-
if constexpr (std::is_same_v<std::decay_t<T>, aod::ChargedJets>) {
145-
selectionObjectPtMin = config.thresholdChargedJetPtMin;
146-
} else if constexpr (std::is_same_v<std::decay_t<T>, aod::NeutralJets>) {
147-
selectionObjectPtMin = config.thresholdNeutralJetPtMin;
148-
} else if constexpr (std::is_same_v<std::decay_t<T>, aod::FullJets>) {
149-
selectionObjectPtMin = config.thresholdFullJetPtMin;
150-
} else if constexpr (std::is_same_v<std::decay_t<T>, aod::JTracks>) {
141+
if constexpr (std::is_same_v<std::decay_t<T>, aod::JTracks>) {
151142
selectionObjectPtMin = config.thresholdTriggerTrackPtMin;
152143
} else if constexpr (std::is_same_v<std::decay_t<T>, aod::JClusters>) {
153144
selectionObjectPtMin = config.thresholdClusterEnergyMin;
@@ -182,9 +173,6 @@ struct RhoEstimatorTask {
182173
}
183174
PROCESS_SWITCH(RhoEstimatorTask, processSetupCollisionSelection, "setup the writing for data based on collisions", false);
184175
PROCESS_SWITCH(RhoEstimatorTask, processSetupEventTriggering, "process software triggers", false);
185-
PROCESS_SWITCH_FULL(RhoEstimatorTask, processSelectionObjects<aod::ChargedJets>, processSelectingChargedJets, "process charged jets", false);
186-
PROCESS_SWITCH_FULL(RhoEstimatorTask, processSelectionObjects<aod::NeutralJets>, processSelectingNeutralJets, "process neutral jets", false);
187-
PROCESS_SWITCH_FULL(RhoEstimatorTask, processSelectionObjects<aod::FullJets>, processSelectingFullJets, "process full jets", false);
188176
PROCESS_SWITCH_FULL(RhoEstimatorTask, processSelectionObjects<aod::JClusters>, processSelectingClusters, "process EMCal clusters", false);
189177
PROCESS_SWITCH_FULL(RhoEstimatorTask, processSelectionObjects<aod::JTracks>, processSelectingTracks, "process high pt tracks", false);
190178

PWGJE/Tasks/jetSubstructureHFOutput.cxx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -695,11 +695,10 @@ struct JetSubstructureHFOutputTask {
695695
PROCESS_SWITCH(JetSubstructureHFOutputTask, processOutputSubstructureMatchingMC, "jet substructure matching output MC", false);
696696

697697
void processOutputJetsMCD(aod::JetCollisionMCD const& collision,
698-
aod::JetMcCollisions const&,
699698
JetTableMCD const& jets,
700699
soa::Join<CandidateTableMCD, CandidateRhosTable> const& candidates)
701700
{
702-
analyseCharged<false>(collision, jets, candidates, products.collisionOutputTableMCD, products.jetOutputTableMCD, products.jetSubstructureOutputTableMCD, splittingMatchesGeoVecVecMCD, splittingMatchesPtVecVecMCD, splittingMatchesHFVecVecMCD, pairMatchesVecVecMCD, jetMappingMCD, candidateMapping, configs.jetPtMinMCD, collision.mcCollision().weight());
701+
analyseCharged<false>(collision, jets, candidates, products.collisionOutputTableMCD, products.jetOutputTableMCD, products.jetSubstructureOutputTableMCD, splittingMatchesGeoVecVecMCD, splittingMatchesPtVecVecMCD, splittingMatchesHFVecVecMCD, pairMatchesVecVecMCD, jetMappingMCD, candidateMapping, configs.jetPtMinMCD, collision.weight());
703702
}
704703
PROCESS_SWITCH(JetSubstructureHFOutputTask, processOutputJetsMCD, "hf jet substructure output MCD", false);
705704

PWGJE/Tasks/jetSubstructureOutput.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -381,10 +381,10 @@ struct JetSubstructureOutputTask {
381381
}
382382
PROCESS_SWITCH(JetSubstructureOutputTask, processOutputSubstructureMatchingMC, "substructure matching output MC", false);
383383

384-
void processOutputMCD(soa::Join<aod::JetCollisionsMCD, aod::BkgChargedRhos>::iterator const& collision, aod::JetMcCollisions const&,
384+
void processOutputMCD(soa::Join<aod::JetCollisionsMCD, aod::BkgChargedRhos>::iterator const& collision,
385385
soa::Join<aod::ChargedMCDetectorLevelJets, aod::ChargedMCDetectorLevelJetConstituents, aod::CMCDJetSSs> const& jets)
386386
{
387-
analyseCharged<false>(collision, jets, collisionOutputTableMCD, jetOutputTableMCD, jetSubstructureOutputTableMCD, splittingMatchesGeoVecVecMCD, splittingMatchesPtVecVecMCD, splittingMatchesHFVecVecMCD, pairMatchesVecVecMCD, jetMappingMCD, jetPtMinMCD, collision.mcCollision().weight());
387+
analyseCharged<false>(collision, jets, collisionOutputTableMCD, jetOutputTableMCD, jetSubstructureOutputTableMCD, splittingMatchesGeoVecVecMCD, splittingMatchesPtVecVecMCD, splittingMatchesHFVecVecMCD, pairMatchesVecVecMCD, jetMappingMCD, jetPtMinMCD, collision.weight());
388388
}
389389
PROCESS_SWITCH(JetSubstructureOutputTask, processOutputMCD, "jet substructure output MCD", false);
390390

0 commit comments

Comments
 (0)