Skip to content

Commit 3e3ab5b

Browse files
authored
Fix o2 linter errors + warnings
1 parent 63d4979 commit 3e3ab5b

File tree

1 file changed

+20
-17
lines changed

1 file changed

+20
-17
lines changed

PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717

1818
#include "PWGCF/Core/CorrelationContainer.h"
1919
#include "PWGCF/Core/PairCuts.h"
20-
#include "PWGCF/TwoParticleCorrelations/DataModel/longrangeDerived.h"
20+
#include "PWGCF/TwoParticleCorrelations/DataModel/LongRangeDerived.h"
2121
#include "PWGLF/DataModel/LFStrangenessTables.h"
2222
#include "PWGMM/Mult/DataModel/bestCollisionTable.h"
2323

@@ -67,7 +67,10 @@ using namespace o2::aod::fwdtrack;
6767
using namespace o2::aod::evsel;
6868
using namespace o2::constants::math;
6969

70-
auto static constexpr kMinFt0cCell = 96;
70+
auto static constexpr KminFt0cCell = 96;
71+
auto static constexpr PionTrackN = 1;
72+
auto static constexpr KaonTrackN = 2;
73+
auto static constexpr ProtonTrackN = 3;
7174
AxisSpec axisEvent{15, 0.5, 15.5, "#Event", "EventAxis"};
7275

7376
enum KindOfParticles {
@@ -243,11 +246,11 @@ struct LongrangeMaker {
243246
if (!myTrackFilter.IsSelected(track))
244247
continue;
245248
tracksLRTable(collisionLRTable.lastIndex(), track.pt(), track.eta(), track.phi(), aod::LRCorrTrkTable::kSpCharge);
246-
if (getTrackPID(track) == 1)
249+
if (getTrackPID(track) == PionTrackN)
247250
tracksLRTable(collisionLRTable.lastIndex(), track.pt(), track.eta(), track.phi(), aod::LRCorrTrkTable::kSpPion);
248-
if (getTrackPID(track) == 2)
251+
if (getTrackPID(track) == KaonTrackN)
249252
tracksLRTable(collisionLRTable.lastIndex(), track.pt(), track.eta(), track.phi(), aod::LRCorrTrkTable::kSpKaon);
250-
if (getTrackPID(track) == 3)
253+
if (getTrackPID(track) == ProtonTrackN)
251254
tracksLRTable(collisionLRTable.lastIndex(), track.pt(), track.eta(), track.phi(), aod::LRCorrTrkTable::kSpProton);
252255
}
253256

@@ -310,16 +313,16 @@ struct LongrangeMaker {
310313
}
311314

312315
// Lambda and Anti-Lambda
313-
bool LambdaTag = isSelectLambda<KindOfV0::kLambda>(col, v0);
314-
bool ALambdaTag = isSelectLambda<KindOfV0::kAntiLambda>(col, v0);
316+
bool lambdaTag = isSelectLambda<KindOfV0::kLambda>(col, v0);
317+
bool antilambdaTag = isSelectLambda<KindOfV0::kAntiLambda>(col, v0);
315318

316319
// Note: candidate compatible with Lambda and Anti-Lambda hypothesis are counted twice (once for each hypothesis)
317-
if (LambdaTag) { // candidate is Lambda
320+
if (lambdaTag) { // candidate is Lambda
318321
massV0 = v0.mLambda();
319322
v0LRTable(collisionLRTable.lastIndex(), posTrack.globalIndex(), negTrack.globalIndex(),
320323
v0.pt(), v0.eta(), v0.phi(), massV0, aod::LRCorrTrkTable::kSpLambda);
321324
}
322-
if (ALambdaTag) { // candidate is Anti-lambda
325+
if (antilambdaTag) { // candidate is Anti-lambda
323326
massV0 = v0.mAntiLambda();
324327
v0LRTable(collisionLRTable.lastIndex(), posTrack.globalIndex(), negTrack.globalIndex(),
325328
v0.pt(), v0.eta(), v0.phi(), massV0, aod::LRCorrTrkTable::kSpALambda);
@@ -461,7 +464,7 @@ struct LongrangeMaker {
461464
auto x = chPos.X() + (*offsetFT0)[i].getX();
462465
auto y = chPos.Y() + (*offsetFT0)[i].getY();
463466
auto z = chPos.Z() + (*offsetFT0)[i].getZ();
464-
if (chno >= kMinFt0cCell)
467+
if (chno >= KminFt0cCell)
465468
z = -z;
466469
auto r = std::sqrt(x * x + y * y);
467470
auto theta = std::atan2(r, z);
@@ -513,7 +516,7 @@ struct LongrangeMaker {
513516
const auto& posTrack = v0.template posTrack_as<TrksTable>();
514517
const auto& negTrack = v0.template negTrack_as<TrksTable>();
515518

516-
float CtauK0s = v0.distovertotmom(col.posX(), col.posY(), col.posZ()) * o2::constants::physics::MassK0;
519+
float ctauK0s = v0.distovertotmom(col.posX(), col.posY(), col.posZ()) * o2::constants::physics::MassK0;
517520

518521
if (v0.mK0Short() < cfgv0trksel.minK0sMass || v0.mK0Short() > cfgv0trksel.maxK0sMass) {
519522
return false;
@@ -530,13 +533,13 @@ struct LongrangeMaker {
530533
if (v0.dcaV0daughters() > cfgv0trksel.maxDcaV0DauK0s) {
531534
return false;
532535
}
533-
if (std::abs(CtauK0s) > cfgv0trksel.maxK0sLifeTime) {
536+
if (std::abs(ctauK0s) > cfgv0trksel.maxK0sLifeTime) {
534537
return false;
535538
}
536539
if (((std::abs(posTrack.tpcNSigmaPi()) > cfgv0trksel.daughPIDCuts) || (std::abs(negTrack.tpcNSigmaPi()) > cfgv0trksel.daughPIDCuts))) {
537540
return false;
538541
}
539-
if ((TMath::Abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPiK0s || TMath::Abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPiK0s)) {
542+
if ((std::abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPiK0s || std::abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPiK0s)) {
540543
return false;
541544
}
542545
return true;
@@ -547,7 +550,7 @@ struct LongrangeMaker {
547550
{
548551
const auto& posTrack = v0.template posTrack_as<TrksTable>();
549552
const auto& negTrack = v0.template negTrack_as<TrksTable>();
550-
float CtauLambda = v0.distovertotmom(col.posX(), col.posY(), col.posZ()) * o2::constants::physics::MassLambda;
553+
float ctauLambda = v0.distovertotmom(col.posX(), col.posY(), col.posZ()) * o2::constants::physics::MassLambda;
551554
if ((v0.mLambda() < cfgv0trksel.minLambdaMass || v0.mLambda() > cfgv0trksel.maxLambdaMass) &&
552555
(v0.mAntiLambda() < cfgv0trksel.minLambdaMass || v0.mAntiLambda() > cfgv0trksel.maxLambdaMass)) {
553556
return false;
@@ -561,10 +564,10 @@ struct LongrangeMaker {
561564
if (v0.dcaV0daughters() > cfgv0trksel.maxDcaV0DauLambda) {
562565
return false;
563566
}
564-
if (pid == KindOfV0::kLambda && (TMath::Abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPr || TMath::Abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPiLambda)) {
567+
if (pid == KindOfV0::kLambda && (std::abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPr || std::abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPiLambda)) {
565568
return false;
566569
}
567-
if (pid == KindOfV0::kAntiLambda && (TMath::Abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPiLambda || TMath::Abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPr)) {
570+
if (pid == KindOfV0::kAntiLambda && (std::abs(v0.dcapostopv()) < cfgv0trksel.minV0DcaPiLambda || std::abs(v0.dcanegtopv()) < cfgv0trksel.minV0DcaPr)) {
568571
return false;
569572
}
570573
if (pid == KindOfV0::kLambda && ((std::abs(posTrack.tpcNSigmaPr()) > cfgv0trksel.daughPIDCuts) || (std::abs(negTrack.tpcNSigmaPi()) > cfgv0trksel.daughPIDCuts))) {
@@ -573,7 +576,7 @@ struct LongrangeMaker {
573576
if (pid == KindOfV0::kAntiLambda && ((std::abs(posTrack.tpcNSigmaPi()) > cfgv0trksel.daughPIDCuts) || (std::abs(negTrack.tpcNSigmaPr()) > cfgv0trksel.daughPIDCuts))) {
574577
return false;
575578
}
576-
if (std::abs(CtauLambda) > cfgv0trksel.maxLambdaLifeTime) {
579+
if (std::abs(ctauLambda) > cfgv0trksel.maxLambdaLifeTime) {
577580
return false;
578581
}
579582
return true;

0 commit comments

Comments
 (0)