Skip to content

[PWGEM,PWGEM-36] PM: add config to store all daughters of all interes… #15975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mhemmer-cern merged 1 commit into from
Apr 27, 2026
+110 −110
Merged

[PWGEM,PWGEM-36] PM: add config to store all daughters of all interes… #15975

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
219 changes: 109 additions & 110 deletions PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2019-2020 CERN and copyright holders of ALICE O2.

Check failure on line 1 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/workflow-file] 

Name of a workflow file must match the name of the main struct in it (without the PWG prefix). (Class implementation files should be in "Core" directories.)

Check failure on line 1 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/o2-task] 

Specify task name only when it cannot be derived from the struct name. Only append to the default name.
// See https://alice-o2.web.cern.ch/copyright for details of the copyright holders.
// All rights not expressly granted are reserved.
//
Expand All @@ -8,13 +8,10 @@
// In applying this license CERN does not waive the privileges and immunities
// granted to it by virtue of its status as an Intergovernmental Organization
// or submit itself to any jurisdiction.
///

/// \file associateMCinfoPhoton.cxx
///
/// \brief This code produces reduced events for photon analyses
///
/// \author Daiki Sekihata (daiki.sekihata@cern.ch)
///

#include "PWGEM/PhotonMeson/DataModel/GammaTablesRedux.h"
#include "PWGEM/PhotonMeson/DataModel/gammaTables.h"
Expand Down Expand Up @@ -46,6 +43,7 @@
#include <cstdlib>
#include <map>
#include <string_view>
#include <unordered_map>
#include <vector>

using namespace o2;
Expand All @@ -60,6 +58,11 @@
using FwdTracksMC = soa::Join<aod::FwdTracks, aod::McFwdTrackLabels>;
using MyEMCClusters = soa::Join<aod::MinClusters, aod::EMCClusterMCLabels>;

struct Counter {
int particles{0};
int events{0};
};

struct AssociateMCInfoPhoton {
enum SubSystem {
kPCM = 0x1,
Expand All @@ -77,11 +80,14 @@

Produces<o2::aod::BinnedGenPts> binnedGenPt;

Configurable<float> max_eta_gen_secondary{"max_eta_gen_secondary", 0.9, "max eta to store generated information"};

Check failure on line 83 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
Configurable<float> margin_z_gen{"margin_z_gen", 15.f, "margin for Z of true photon conversion point to store generated information"};

Check failure on line 84 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
Configurable<float> max_rxy_gen{"max_rxy_gen", 100, "max rxy to store generated information"};

Check failure on line 85 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
Configurable<bool> requireGammaGammaDecay{"requireGammaGammaDecay", false, "require gamma gamma decay for generated pi0 and eta meson"};
Configurable<float> cfg_max_eta_photon{"cfg_max_eta_gamma", 0.8, "max eta for photons at PV"};

Check failure on line 87 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
Configurable<float> maxPt{"maxPt", 20.f, "max pT for BinnedGenPts table"};
Configurable<float> maxY{"maxY", 0.9f, "max |rapidity| for BinnedGenPts table"};
Configurable<bool> doStoreAllDaughters{"doStoreAllDaughters", false, "flag to enable storing of all photon, pi0, eta, eta' and omega daughters. This will increase the dervied data size!"};

HistogramRegistry registry{"EMMCEvent"};

Expand Down Expand Up @@ -136,16 +142,72 @@
std::vector<uint16_t> genPi0; // primary, pt, y
std::vector<uint16_t> genEta; // primary, pt, y

template <o2::soa::is_iterator TMCParticle, o2::soa::is_iterator TMCDaughter>
void selectDaughtersToStore(TMCParticle& particle, int64_t mcParticleSize, TMCDaughter& daughterIter, int eventIdx, std::unordered_map<uint64_t, int>& fNewLabels, std::map<uint64_t, int>& fNewLabelsReversed, std::unordered_map<uint64_t, int>& fEventIdx, Counter& fCounter)
{
if (!particle.has_daughters()) {
return;
}
for (int daughterId = particle.daughtersIds()[0]; daughterId <= particle.daughtersIds()[1]; ++daughterId) {
if (daughterId < mcParticleSize) {
daughterIter.setCursor(daughterId);
if (!fNewLabels.contains(daughterIter.globalIndex())) {
fNewLabels[daughterIter.globalIndex()] = fCounter.particles;
fNewLabelsReversed[fCounter.particles] = daughterIter.globalIndex();
fEventIdx[daughterIter.globalIndex()] = eventIdx;
fCounter.particles++;
}
}
}
}

template <o2::soa::is_iterator TMCParticle, o2::soa::is_iterator TMCCollision>
void selectMothersToStore(int motherId, int64_t mcParticleSize, TMCParticle& motherParticle, TMCParticle& daughterIter, TMCCollision const& mcCollisionIter, std::unordered_map<uint64_t, int>& fNewLabels, std::map<uint64_t, int>& fNewLabelsReversed, std::unordered_map<uint64_t, int>& fEventIdx, std::unordered_map<uint64_t, int>& fEventLabels, Counter& fCounter)
{
while (motherId > -1) {
if (motherId < mcParticleSize) { // protect against bad mother indices. why is this needed?
motherParticle.setCursor(motherId);
int eventIdx = fEventLabels.find(mcCollisionIter.globalIndex())->second;

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!fNewLabels.contains(motherParticle.globalIndex())) {
fNewLabels[motherParticle.globalIndex()] = fCounter.particles;
fNewLabelsReversed[fCounter.particles] = motherParticle.globalIndex();
fEventIdx[motherParticle.globalIndex()] = eventIdx;
fCounter.particles++;
}
// If this ancestor is a photon, pi0, eta, omega or eta prime store all its daughters
if ((doStoreAllDaughters) && (std::abs(motherParticle.pdgCode()) == PDG_t::kGamma ||
std::abs(motherParticle.pdgCode()) == PDG_t::kPi0 ||
std::abs(motherParticle.pdgCode()) == o2::constants::physics::Pdg::kEta ||
std::abs(motherParticle.pdgCode()) == o2::constants::physics::Pdg::kEtaPrime ||
std::abs(motherParticle.pdgCode()) == o2::constants::physics::Pdg::kOmega)) {
selectDaughtersToStore(motherParticle, mcParticleSize, daughterIter,
eventIdx, fNewLabels, fNewLabelsReversed,
fEventIdx, fCounter);
}

if (motherParticle.has_mothers()) {
motherId = motherParticle.mothersIds()[0]; // first mother index
} else {
motherId = -999;
}
} else {
motherId = -999;
}
} // end of mother chain loop
}

template <uint8_t system, typename TTracks, typename TFwdTracks, typename TPCMs, typename TPCMLegs, typename TPHOSs, typename TEMCs, typename TEMPrimaryElectrons>
void skimmingMC(MyCollisionsMC const& collisions, aod::BCs const&, aod::McCollisions const& mcCollisions, aod::McParticles const& mcParticles, TTracks const& o2tracks, TFwdTracks const&, TPCMs const& v0photons, TPCMLegs const& legs, TPHOSs const&, TEMCs const& emcclusters, TEMPrimaryElectrons const& emprimaryelectrons)
{
// temporary variables used for the indexing of the skimmed MC stack
std::map<uint64_t, int> fNewLabels;
std::unordered_map<uint64_t, int> fNewLabels;
std::map<uint64_t, int> fNewLabelsReversed;
// std::map<uint64_t, uint16_t> fMCFlags;
std::map<uint64_t, int> fEventIdx;
std::map<uint64_t, int> fEventLabels;
int fCounters[2] = {0, 0}; //! [0] - particle counter, [1] - event counter
std::unordered_map<uint64_t, int> fEventIdx;
std::unordered_map<uint64_t, int> fEventLabels;
Counter fCounter{.particles = 0, .events = 0}; //! [0] - particle counter, [1] - event counter
auto hBinFinder = registry.get<TH2>(HIST("Generated/h2PtY_Gamma"));

// collision iterator from EMCal cluster
Expand All @@ -156,6 +218,7 @@

// mc particles iterator for mother
auto motherParticle = mcParticles.begin();
auto daughterIter = mcParticles.begin();

// mc particles iterator for mother and other mc particles
auto mcParticleIter = mcParticles.begin();
Expand Down Expand Up @@ -186,7 +249,7 @@
auto groupedMcParticles = mcParticles.sliceBy(perMcCollision, mcCollisionIter.globalIndex());

for (const auto& mcParticle : groupedMcParticles) { // store necessary information for denominator of efficiency
if ((mcParticle.isPhysicalPrimary() || mcParticle.producedByGenerator()) && std::fabs(mcParticle.y()) < 0.9f && mcParticle.pt() < 20.f) {
if ((mcParticle.isPhysicalPrimary() || mcParticle.producedByGenerator()) && std::fabs(mcParticle.y()) < maxY && mcParticle.pt() < maxPt) {
auto binNumber = hBinFinder->FindBin(mcParticle.pt(), std::fabs(mcParticle.y())); // caution: pack

bool isMesonAccepted = false;
Expand Down Expand Up @@ -229,8 +292,8 @@
// make an entry for this MC event only if it was not already added to the table
if (!(fEventLabels.find(mcCollisionIter.globalIndex()) != fEventLabels.end())) {
mcevents(mcCollisionIter.globalIndex(), mcCollisionIter.generatorsID(), mcCollisionIter.posX(), mcCollisionIter.posY(), mcCollisionIter.posZ(), mcCollisionIter.impactParameter(), mcCollisionIter.eventPlaneAngle());
fEventLabels[mcCollisionIter.globalIndex()] = fCounters[1];
fCounters[1]++;
fEventLabels[mcCollisionIter.globalIndex()] = fCounter.events;
fCounter.events++;
binnedGenPt(genGamma, genPi0, genEta);
}

Expand All @@ -238,11 +301,11 @@
mceventlabels(fEventLabels.find(mcCollisionIter.globalIndex())->second, collision.mcMask());

for (const auto& mcParticle : groupedMcParticles) { // store necessary information for denominator of efficiency
if (mcParticle.pt() < 1e-3 || std::fabs(mcParticle.vz()) > 250 || std::sqrt(std::pow(mcParticle.vx(), 2) + std::pow(mcParticle.vy(), 2)) > max_rxy_gen) {

Check failure on line 304 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
continue;
}
int pdg = mcParticle.pdgCode();
if (std::abs(pdg) > 1e+9) {

Check failure on line 308 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
continue;
}

Expand All @@ -259,21 +322,21 @@

if (motherParticle.pdgCode() == PDG_t::kGamma && (motherParticle.isPhysicalPrimary() || motherParticle.producedByGenerator()) && std::fabs(motherParticle.eta()) < max_eta_gen_secondary) {
// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(mcParticle.globalIndex()) != fNewLabels.end())) { // store electron information. !!Not photon!!
fNewLabels[mcParticle.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = mcParticle.globalIndex();
if (!fNewLabels.contains(mcParticle.globalIndex())) { // store electron information. !!Not photon!!
fNewLabels[mcParticle.globalIndex()] = fCounter.particles;
fNewLabelsReversed[fCounter.particles] = mcParticle.globalIndex();
// fMCFlags[mcParticle.globalIndex()] = mcflags;
fEventIdx[mcParticle.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
fCounter.particles++;
}

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(motherParticle.globalIndex()) != fNewLabels.end())) { // store conversion photon
fNewLabels[motherParticle.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = motherParticle.globalIndex();
if (!fNewLabels.contains(motherParticle.globalIndex())) { // store conversion photon
fNewLabels[motherParticle.globalIndex()] = fCounter.particles;
fNewLabelsReversed[fCounter.particles] = motherParticle.globalIndex();
// fMCFlags[motherParticle.globalIndex()] = mcflags;
fEventIdx[motherParticle.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
fCounter.particles++;
}
}
}
Expand Down Expand Up @@ -312,42 +375,20 @@
// LOGF(info, "mcParticleIter.globalIndex() = %d, mcParticleIter.index() = %d", mcParticleIter.globalIndex(), mcParticleIter.index()); // these are exactly the same.

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(mcParticleIter.globalIndex()) != fNewLabels.end())) {
fNewLabels[mcParticleIter.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = mcParticleIter.globalIndex();
// fMCFlags[mcParticleIter.globalIndex()] = mcflags;
auto [iter, isNew] = fNewLabels.try_emplace(mcParticleIter.globalIndex(), fCounter.particles);
if (isNew) {
fNewLabelsReversed[fCounter.particles] = mcParticleIter.globalIndex();
fEventIdx[mcParticleIter.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
fCounter.particles++;
}
v0legmclabels(fNewLabels.find(mcParticleIter.index())->second, o2TrackIter.mcMask());
v0legmclabels(iter->second, o2TrackIter.mcMask());

// Next, store mother-chain of this reconstructed track.
int motherid = -999; // first mother index
if (mcParticleIter.has_mothers()) {
motherid = mcParticleIter.mothersIds()[0]; // first mother index
}
while (motherid > -1) {
if (motherid < mcParticles.size()) { // protect against bad mother indices. why is this needed?
motherParticle.setCursor(motherid);

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(motherParticle.globalIndex()) != fNewLabels.end())) {
fNewLabels[motherParticle.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = motherParticle.globalIndex();
// fMCFlags[motherParticle.globalIndex()] = mcflags;
fEventIdx[motherParticle.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
}

if (motherParticle.has_mothers()) {
motherid = motherParticle.mothersIds()[0]; // first mother index
} else {
motherid = -999;
}
} else {
motherid = -999;
}
} // end of mother chain loop
selectMothersToStore(motherid, mcParticles.size(), motherParticle, daughterIter, mcCollisionIter, fNewLabels, fNewLabelsReversed, fEventIdx, fEventLabels, fCounter);
} // end of leg loop
} // end of v0 loop
}
Expand All @@ -369,43 +410,20 @@
mcParticleIter.setCursor(o2TrackIter.mcParticleId());

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(mcParticleIter.globalIndex()) != fNewLabels.end())) {
fNewLabels[mcParticleIter.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = mcParticleIter.globalIndex();
// fMCFlags[mcParticleIter.globalIndex()] = mcflags;
auto [iter, isNew] = fNewLabels.try_emplace(mcParticleIter.globalIndex(), fCounter.particles);
if (isNew) {
fNewLabelsReversed[fCounter.particles] = mcParticleIter.globalIndex();
fEventIdx[mcParticleIter.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
fCounter.particles++;
}
emprimaryelectronmclabels(fNewLabels.find(mcParticleIter.index())->second, o2TrackIter.mcMask());
emprimaryelectronmclabels(iter->second, o2TrackIter.mcMask());

// Next, store mother-chain of this reconstructed track.
int motherid = -999; // first mother index
if (mcParticleIter.has_mothers()) {
motherid = mcParticleIter.mothersIds()[0]; // first mother index
}
while (motherid > -1) {
if (motherid < mcParticles.size()) { // protect against bad mother indices. why is this needed?
motherParticle.setCursor(motherid);

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(motherParticle.globalIndex()) != fNewLabels.end())) {
fNewLabels[motherParticle.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = motherParticle.globalIndex();
// fMCFlags[motherParticle.globalIndex()] = mcflags;
fEventIdx[motherParticle.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
}

if (motherParticle.has_mothers()) {
motherid = motherParticle.mothersIds()[0]; // first mother index
} else {
motherid = -999;
}
} else {
motherid = -999;
}
} // end of mother chain loop

selectMothersToStore(motherid, mcParticles.size(), motherParticle, daughterIter, mcCollisionIter, fNewLabels, fNewLabelsReversed, fEventIdx, fEventLabels, fCounter);
} // end of em primary electron loop
}

Expand All @@ -430,42 +448,21 @@
mcPhoton.setCursor(emcParticleId);

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(mcPhoton.globalIndex()) != fNewLabels.end())) {
fNewLabels[mcPhoton.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = mcPhoton.globalIndex();
auto [iter, isNew] = fNewLabels.try_emplace(mcPhoton.globalIndex(), fCounter.particles);
if (isNew) {
fNewLabelsReversed[fCounter.particles] = mcPhoton.globalIndex();
fEventIdx[mcPhoton.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
fCounter.particles++;
}
vEmcMcParticleIds.emplace_back(fNewLabels.find(mcPhoton.index())->second);
vEmcMcParticleIds.emplace_back(iter->second);
// ememcclustermclabels(fNewLabels.find(mcPhoton.index())->second);

// Next, store mother-chain of this reconstructed track.
int motherid = -999; // first mother index
if (mcPhoton.has_mothers()) {
motherid = mcPhoton.mothersIds()[0]; // first mother index
}
while (motherid > -1) {
if (motherid < mcParticles.size()) { // protect against bad mother indices. why is this needed?
motherParticle.setCursor(motherid);

// if the MC truth particle corresponding to this reconstructed track which is not already written, add it to the skimmed MC stack
if (!(fNewLabels.find(motherParticle.globalIndex()) != fNewLabels.end())) {
fNewLabels[motherParticle.globalIndex()] = fCounters[0];
fNewLabelsReversed[fCounters[0]] = motherParticle.globalIndex();
fEventIdx[motherParticle.globalIndex()] = fEventLabels.find(mcCollisionIter.globalIndex())->second;
fCounters[0]++;
}

if (motherParticle.has_mothers()) {
motherid = motherParticle.mothersIds()[0]; // first mother index
} else {
motherid = -999;
}
} else {
motherid = -999;
}
} // end of mother chain loop

selectMothersToStore(motherid, mcParticles.size(), motherParticle, daughterIter, mcCollisionIter, fNewLabels, fNewLabelsReversed, fEventIdx, fEventLabels, fCounter);
} // end of loop over mc particles of the current emc cluster
ememcclustermclabels(vEmcMcParticleIds);

Expand All @@ -481,8 +478,9 @@
if (mcParticleIter.has_mothers()) {
for (const auto& m : mcParticleIter.mothersIds()) {
if (m < mcParticles.size()) { // protect against bad mother indices
if (fNewLabels.find(m) != fNewLabels.end()) {
mothers.push_back(fNewLabels.find(m)->second);
auto iter = fNewLabels.find(m);
if (iter != fNewLabels.end()) {
mothers.push_back(iter->second);
}
} else {
LOG(info) << "Mother label (" << m << ") exceeds the McParticles size (" << mcParticles.size() << ")";
Expand All @@ -500,8 +498,9 @@
if (d < mcParticles.size()) { // protect against bad daughter indices
// auto dau_tmp = mcParticles.iteratorAt(d);
// LOGF(info, "daughter pdg = %d", dau_tmp.pdgCode());
if (fNewLabels.find(d) != fNewLabels.end()) {
daughters.push_back(fNewLabels.find(d)->second);
auto iter = fNewLabels.find(d);
if (iter != fNewLabels.end()) {
daughters.push_back(iter->second);
}
} else {
LOG(error) << "Daughter label (" << d << ") exceeds the McParticles size (" << mcParticles.size() << ")";
Expand All @@ -521,14 +520,14 @@
// fMCFlags.clear();
fEventIdx.clear();
fEventLabels.clear();
fCounters[0] = 0;
fCounters[1] = 0;
fCounter.particles = 0;
fCounter.events = 0;
} // end of skimmingMC

template <typename Daughters>
template <o2::soa::is_table Daughters>
inline bool areTwoPhotonDaughtersAccepted(const Daughters& lDaughters, float maxEta)
{
if (lDaughters.size() != 2) {

Check failure on line 530 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
return false;
}

Expand Down Expand Up @@ -628,5 +627,5 @@
WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
{
return WorkflowSpec{
adaptAnalysisTask<AssociateMCInfoPhoton>(cfgc, TaskName{"associate-mc-info-photon"})};

Check failure on line 630 in PWGEM/PhotonMeson/TableProducer/associateMCinfoPhoton.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/o2-task] 

Device names associate-mc-info-photon and associate-m-c-info-photon generated from the specified task name associate-mc-info-photon and from the struct name AssociateMCInfoPhoton, respectively, differ in hyphenation. Consider fixing capitalisation of the struct name to AssociateMcInfoPhoton and removing TaskName.
}
Loading
Loading