Skip to content

Commit fa1066a

Browse files
authored
[PWGEM,Photon] Remove unnecessary registry argument from task function (#17827)
1 parent a0ba873 commit fa1066a

4 files changed

Lines changed: 20 additions & 20 deletions

File tree

PWGEM/PhotonMeson/Core/TaggingPi0MC.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ using MyMCCollision = MyMCCollisions::iterator;
7777
using MyV0Photons = o2::soa::Join<o2::aod::V0PhotonsKF, o2::aod::V0KFEMEventIds>;
7878
using MyV0Photon = MyV0Photons::iterator;
7979

80-
using MyEMCClusters = o2::soa::Join<o2::aod::SkimEMCClusters, o2::aod::EMEMCClusterMCLabels, o2::aod::EMCEMEventIds>;
80+
using MyEMCClusters = o2::soa::Join<o2::aod::SkimEMCClusters, o2::aod::EMEMCClusterMCLabels_001, o2::aod::EMCEMEventIds>;
8181
using MyEMCCluster = MyEMCClusters::iterator;
8282

8383
// using MyPHOSClusters = o2::soa::Join<o2::aod::PHOSClusters, o2::aod::PHOSEMEventIds>;

PWGEM/PhotonMeson/Tasks/Pi0EtaToGammaGammaMCEMCEMC.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ using namespace o2::aod;
2727
using namespace o2::framework;
2828
using namespace o2::aod::pwgem::photonmeson::photonpair;
2929

30-
using MyEMCClusters = soa::Join<aod::MinClusters, aod::EMEMCClusterMCLabels, aod::EMCEMEventIds>;
30+
using MyEMCClusters = soa::Join<aod::MinClusters, aod::EMEMCClusterMCLabels_001, aod::EMCEMEventIds>;
3131

3232
WorkflowSpec defineDataProcessing(ConfigContext const& context)
3333
{

PWGEM/PhotonMeson/Tasks/emcalMcTask.cxx

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ struct EmcalMcTask {
133133

134134
SliceCache cache;
135135

136-
using EMCalPhotons = soa::Join<aod::EMCEMEventIds, aod::MinClusters, aod::EMEMCClusterMCLabels>;
136+
using EMCalPhotons = soa::Join<aod::EMCEMEventIds, aod::MinClusters, aod::EMEMCClusterMCLabels_001>;
137137

138138
using Colls = soa::Join<aod::PMEvents, aod::EMEventsAlias, aod::EMEventsMult_000, aod::EMEventsCent_000, aod::EMMCEventLabels, aod::EmMagFields>;
139139

@@ -297,13 +297,13 @@ struct EmcalMcTask {
297297

298298
// One templated fill function instead of 9 copy-pasted blocks
299299
template <const int type, o2::soa::is_iterator TCluster, o2::soa::is_iterator TMC>
300-
void fillClusterHistos(HistogramRegistry& histRegistry, TCluster const& clu, TMC const& mcPart, float centOrMult)
300+
void fillClusterHistos(TCluster const& cluster, TMC const& mcPart, float centOrMult)
301301
{
302-
static constexpr std::string_view subDir = kSubDirs[type];
302+
static constexpr std::string_view SubDir = kSubDirs[type];
303303

304-
histRegistry.fill(HIST(subDir) + HIST("hM02"), clu.m02(), clu.e(), centOrMult);
305-
histRegistry.fill(HIST(subDir) + HIST("hEtaRel"), clu.eta() - mcPart.eta(), clu.e(), centOrMult);
306-
histRegistry.fill(HIST(subDir) + HIST("hPhiRel"), clu.phi() - mcPart.phi(), clu.e(), centOrMult);
304+
registry.fill(HIST(SubDir) + HIST("hM02"), cluster.m02(), cluster.e(), centOrMult);
305+
registry.fill(HIST(SubDir) + HIST("hEtaRel"), cluster.eta() - mcPart.eta(), cluster.e(), centOrMult);
306+
registry.fill(HIST(SubDir) + HIST("hPhiRel"), cluster.phi() - mcPart.phi(), cluster.e(), centOrMult);
307307
}
308308

309309
// PCM-EMCal same event
@@ -340,23 +340,23 @@ struct EmcalMcTask {
340340
mcPhoton1.setCursor(photonEMC.emmcparticleIds()[0]);
341341

342342
if (std::abs(mcPhoton1.pdgCode()) == PDG_t::kGamma) {
343-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPhoton>(registry, photonEMC, mcPhoton1, centOrMult);
344-
} else if (std::abs(mcPhoton1.pdgCode()) == PDG_t::kElectron) {
345-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kElectron>(registry, photonEMC, mcPhoton1, centOrMult);
346-
} else if (mcPhoton1.pdgCode() == -PDG_t::kElectron) {
347-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPositron>(registry, photonEMC, mcPhoton1, centOrMult);
343+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPhoton>(photonEMC, mcPhoton1, centOrMult);
344+
} else if (mcPhoton1.pdgCode() == PDG_t::kElectron) {
345+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kElectron>(photonEMC, mcPhoton1, centOrMult);
346+
} else if (mcPhoton1.pdgCode() == PDG_t::kPositron) {
347+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPositron>(photonEMC, mcPhoton1, centOrMult);
348348
} else if (std::abs(mcPhoton1.pdgCode()) == PDG_t::kPi0) {
349-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPi0>(registry, photonEMC, mcPhoton1, centOrMult);
349+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPi0>(photonEMC, mcPhoton1, centOrMult);
350350
} else if (std::abs(mcPhoton1.pdgCode()) == o2::constants::physics::Pdg::kEta) {
351-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kEta>(registry, photonEMC, mcPhoton1, centOrMult);
351+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kEta>(photonEMC, mcPhoton1, centOrMult);
352352
} else if (std::abs(mcPhoton1.pdgCode()) == o2::constants::physics::Pdg::kOmega) {
353-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kOmega>(registry, photonEMC, mcPhoton1, centOrMult);
353+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kOmega>(photonEMC, mcPhoton1, centOrMult);
354354
} else if (std::abs(mcPhoton1.pdgCode()) == PDG_t::kPiPlus) {
355-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPion>(registry, photonEMC, mcPhoton1, centOrMult);
355+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kPion>(photonEMC, mcPhoton1, centOrMult);
356356
} else if (std::abs(mcPhoton1.pdgCode()) == PDG_t::kKPlus) {
357-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kKaon>(registry, photonEMC, mcPhoton1, centOrMult);
357+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kKaon>(photonEMC, mcPhoton1, centOrMult);
358358
} else {
359-
fillClusterHistos<o2::em::emcal::mc::ParticleType::kOther>(registry, photonEMC, mcPhoton1, centOrMult);
359+
fillClusterHistos<o2::em::emcal::mc::ParticleType::kOther>(photonEMC, mcPhoton1, centOrMult);
360360
}
361361
} // for (const auto& photonEMC : photonsEMCPerCollision) {
362362
}

PWGEM/PhotonMeson/Tasks/photonResoTask.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ struct PhotonResoTask {
177177

178178
SliceCache cache;
179179

180-
using EMCalPhotons = soa::Join<aod::EMCEMEventIds, aod::MinClusters, aod::EMEMCClusterMCLabels>;
180+
using EMCalPhotons = soa::Join<aod::EMCEMEventIds, aod::MinClusters, aod::EMEMCClusterMCLabels_001>;
181181
using PcmPhotons = soa::Join<aod::V0PhotonsKF, aod::V0KFEMEventIds>;
182182

183183
using PcmMcLegs = soa::Join<aod::V0Legs, aod::V0LegMCLabels>;

0 commit comments

Comments
 (0)