linter

Author: EmilGorm

PWGCF/TwoParticleCorrelations/Tasks/twoParticleCorrelationsMpi.cxx

@@ -9,7 +9,7 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
-/// \file twoParticleCorrelationsMPI.cxx
+/// \file twoParticleCorrelationsMpi.cxx
 /// \brief task for the MPI-proxy classification of correlation calculations with CF-filtered tracks for O2 analysis
 /// \author Emil Gorm Dahlbæk Nielsen <emil.gorm.nielsen@cern.ch>
 
@@ -75,7 +75,7 @@ using namespace constants::math;
 
 static constexpr float kCfgPairCutDefaults[1][5] = {{-1, -1, -1, -1, -1}};
 
-struct twoParticleCorrelationsMPI {
+struct twoParticleCorrelationsMpi {
   SliceCache cache;
 
   // Configuration
@@ -770,7 +770,8 @@ struct twoParticleCorrelationsMPI {
   {
     const AxisSpec multAxis(axisMultiplicity);
     const auto& edges = multAxis.binEdges;
-    if (edges.size() < 2 || multiplicity < edges.front() || multiplicity >= edges.back()) {
+    const int minEdges = 2;
+    if (edges.size() < minEdges || multiplicity < edges.front() || multiplicity >= edges.back()) {
       return -1;
     }
 
@@ -780,7 +781,8 @@ struct twoParticleCorrelationsMPI {
 
   int classify(double s) const
   {
-    if (ec.quantileBoundaries.size() < 2) {
+    const int minEdges = 2;
+    if (ec.quantileBoundaries.size() < minEdges) {
       return -1;
     }
 
@@ -862,7 +864,7 @@ struct twoParticleCorrelationsMPI {
     fillQA(collision, multiplicity, tracks);
     fillCorrelations<CorrelationContainer::kCFStepReconstructed>(same, tracks, tracks, multiplicity, collision.posZ(), getMagneticField(bc.timestamp()), 1.0f);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processSameAOD, "Process same event on AOD", true);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processSameAOD, "Process same event on AOD", true);
 
   template <class CollType, class TTracks1, class TTracks2>
   void processSameDerivedT(CollType const& collision, TTracks1 const& tracks1, TTracks2 const& tracks2)
@@ -906,15 +908,15 @@ struct twoParticleCorrelationsMPI {
   {
     processSameDerivedT(collision, tracks, tracks);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processSameDerived, "Process same event on derived data", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processSameDerived, "Process same event on derived data", false);
 
   void processSameDerivedMultSet(soa::Filtered<soa::Join<aod::CFCollisions, aod::CFMultSets>>::iterator const& collision, soa::Filtered<aod::CFTracks> const& tracks)
   {
     if (!passOutlier(collision))
       return;
     processSameDerivedT(collision, tracks, tracks);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processSameDerivedMultSet, "Process same event on derived data with multiplicity sets", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processSameDerivedMultSet, "Process same event on derived data with multiplicity sets", false);
 
   using BinningTypeAOD = ColumnBinningPolicy<aod::collision::PosZ, aod::cent::CentRun2V0M>;
   void processMixedAOD(AodCollisions const& collisions, AodTracks const& tracks, aod::BCsWithTimestamps const&)
@@ -957,7 +959,7 @@ struct twoParticleCorrelationsMPI {
       fillCorrelations<CorrelationContainer::kCFStepReconstructed>(mixed, tracks1, tracks2, collision1.centRun2V0M(), collision1.posZ(), getMagneticField(bc.timestamp()), 1.0f / it.currentWindowNeighbours());
     }
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMixedAOD, "Process mixed events on AOD", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMixedAOD, "Process mixed events on AOD", false);
 
   template <class CollType, typename... TrackTypes>
   void processMixedDerivedT(CollType const& collisions, TrackTypes&&... tracks)
@@ -1029,13 +1031,13 @@ struct twoParticleCorrelationsMPI {
   {
     processMixedDerivedT(collisions, tracks);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMixedDerived, "Process mixed events on derived data", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMixedDerived, "Process mixed events on derived data", false);
 
   void processMixedDerivedMultSet(soa::Filtered<soa::Join<aod::CFCollisions, aod::CFMultSets>> const& collisions, DerivedTracks const& tracks)
   {
     processMixedDerivedT(collisions, tracks);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMixedDerivedMultSet, "Process mixed events on derived data with multiplicity sets", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMixedDerivedMultSet, "Process mixed events on derived data with multiplicity sets", false);
 
   int getSpecies(int pdgCode)
   {
@@ -1103,7 +1105,7 @@ struct twoParticleCorrelationsMPI {
       }
     }
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMCEfficiency, "MC: Extract efficiencies", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMCEfficiency, "MC: Extract efficiencies", false);
 
   template <class Particles1, class Particles2>
   void processMCSameDerivedT(soa::Filtered<aod::CFMcCollisions>::iterator const& mcCollision, Particles1 const& mcParticles1, Particles2 const& mcParticles2, soa::SmallGroups<aod::CFCollisionsWithLabel> const& collisions)
@@ -1154,7 +1156,7 @@ struct twoParticleCorrelationsMPI {
   {
     processMCSameDerivedT(mcCollision, mcParticles, mcParticles, collisions);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMCSameDerived, "Process MC same event on derived data", false);
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMCSameDerived, "Process MC same event on derived data", false);
 
   PresliceUnsorted<aod::CFCollisionsWithLabel> collisionPerMCCollision = aod::cfcollision::cfMcCollisionId;
   template <typename... ParticleTypes>
@@ -1223,12 +1225,12 @@ struct twoParticleCorrelationsMPI {
   {
     processMCMixedDerivedT(mcCollisions, collisions, mcParticles);
   }
-  PROCESS_SWITCH(twoParticleCorrelationsMPI, processMCMixedDerived,
+  PROCESS_SWITCH(twoParticleCorrelationsMpi, processMCMixedDerived,
                  "Process MC mixed events on derived data", false);
 };
 
 WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
 {
   return WorkflowSpec{
-    adaptAnalysisTask<twoParticleCorrelationsMPI>(cfgc)};
+    adaptAnalysisTask<twoParticleCorrelationsMpi>(cfgc)};
 }