Skip to content

Commit 2037dc0

Browse files
author
Prottay Das
committed
o2linter issues handled for this PR
1 parent ad1ae4e commit 2037dc0

File tree

2 files changed

+33
-32
lines changed

2 files changed

+33
-32
lines changed

‎PWGLF/DataModel/LFCKSSpinalignmentTables.h‎

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
// granted to it by virtue of its status as an Intergovernmental Organization
1010
// or submit itself to any jurisdiction.
1111

12+
/// \file cksspinalignment.cxx
13+
/// \brief DataModel for Charged KStar spin alignment
1214
///
1315
/// \author Prottay Das <prottay.das@cern.ch>
1416

@@ -31,7 +33,7 @@ namespace o2::aod
3133
namespace kshortpionevent
3234
{
3335
DECLARE_SOA_COLUMN(Cent, cent, float);
34-
DECLARE_SOA_COLUMN(CollIndex, collindex, float);
36+
DECLARE_SOA_COLUMN(CollIndex, collIndex, float);
3537
DECLARE_SOA_COLUMN(PsiFT0C, psiFT0C, float);
3638
DECLARE_SOA_COLUMN(PsiFT0A, psiFT0A, float);
3739
DECLARE_SOA_COLUMN(PsiTPC, psiTPC, float);
@@ -47,28 +49,27 @@ using KShortpionEvent = KShortpionEvents::iterator;
4749

4850
namespace kshortpionpair
4951
{
50-
DECLARE_SOA_INDEX_COLUMN(KShortpionEvent, kshortpionevent);
52+
DECLARE_SOA_INDEX_COLUMN(KShortpionEventId, kshortpionEventId);
5153
DECLARE_SOA_COLUMN(V0Cospa, v0Cospa, float); //! V0 Cospa
5254
DECLARE_SOA_COLUMN(V0Radius, v0Radius, float); //! V0 Radius
5355
DECLARE_SOA_COLUMN(DcaPositive, dcaPositive, float); //! DCA Positive
5456
DECLARE_SOA_COLUMN(DcaNegative, dcaNegative, float); //! DCA Negative
5557
DECLARE_SOA_COLUMN(DcaBetweenDaughter, dcaBetweenDaughter, float); //! DCA between daughters
5658
DECLARE_SOA_COLUMN(V0Lifetime, v0Lifetime, float); //! KShort lifetime
57-
// DECLARE_SOA_COLUMN(Armenteros, arm, float); //! Armenteros cut
58-
DECLARE_SOA_COLUMN(KShortPx, kshPx, float); //! KShort Px
59-
DECLARE_SOA_COLUMN(KShortPy, kshPy, float); //! KShort Py
60-
DECLARE_SOA_COLUMN(KShortPz, kshPz, float); //! KShort Pz
61-
DECLARE_SOA_COLUMN(KShortMass, kshMass, float); //! KShort Mass
62-
DECLARE_SOA_COLUMN(PionBachPx, pibachPx, float); //! Bachelor Pion Px
63-
DECLARE_SOA_COLUMN(PionBachPy, pibachPy, float); //! Bachelor Pion Py
64-
DECLARE_SOA_COLUMN(PionBachPz, pibachPz, float); //! Bachelor Pion Pz
65-
DECLARE_SOA_COLUMN(PionBachSign, pibachSign, int); //! Bachelor Pion Sign
66-
DECLARE_SOA_COLUMN(PionBachTPC, pibachnsigtpc, float); //! Bachelor Pion nsigmatpc
67-
DECLARE_SOA_COLUMN(PionBachTOFHit, pibachtofhit, int); //! Bachelor Pion tof hit availability
68-
DECLARE_SOA_COLUMN(PionBachTOF, pibachnsigtof, float); //! Bachelor Pion nsigmatof
69-
DECLARE_SOA_COLUMN(PionBachIndex, pibachIndex, int); //! Bachelor Pion index
70-
DECLARE_SOA_COLUMN(PionIndex1, pionIndex1, int); //! Daughter Pion index1
71-
DECLARE_SOA_COLUMN(PionIndex2, pionIndex2, int); //! Daughter Pion index2
59+
DECLARE_SOA_COLUMN(KShortPx, kShortPx, float); //! KShort Px
60+
DECLARE_SOA_COLUMN(KShortPy, kShortPy, float); //! KShort Py
61+
DECLARE_SOA_COLUMN(KShortPz, kShortPz, float); //! KShort Pz
62+
DECLARE_SOA_COLUMN(KShortMass, kShortMass, float); //! KShort Mass
63+
DECLARE_SOA_COLUMN(PionBachPx, pionbachPx, float); //! Bachelor Pion Px
64+
DECLARE_SOA_COLUMN(PionBachPy, pionbachPy, float); //! Bachelor Pion Py
65+
DECLARE_SOA_COLUMN(PionBachPz, pionbachPz, float); //! Bachelor Pion Pz
66+
DECLARE_SOA_COLUMN(PionBachSign, pionbachSign, int); //! Bachelor Pion Sign
67+
DECLARE_SOA_COLUMN(PionBachTPC, pionbachtpc, float); //! Bachelor Pion nsigmatpc
68+
DECLARE_SOA_COLUMN(PionBachTOFHit, pibachtofhit, int); //! Bachelor Pion tof hit availability
69+
DECLARE_SOA_COLUMN(PionBachTOF, pionbachtof, float); //! Bachelor Pion nsigmatof
70+
DECLARE_SOA_COLUMN(PionBachIndex, pionbachIndex, int); //! Bachelor Pion index
71+
DECLARE_SOA_COLUMN(PionIndex1, pionIndex1, int); //! Daughter Pion index1
72+
DECLARE_SOA_COLUMN(PionIndex2, pionIndex2, int); //! Daughter Pion index2
7273
} // namespace kshortpionpair
7374
DECLARE_SOA_TABLE(KShortpionPairs, "AOD", "KSHORTPIONPAIR",
7475
o2::soa::Index<>,

‎PWGLF/TableProducer/Resonances/cksspinalignment.cxx‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -84,19 +84,19 @@ struct cksspinalignment {
8484

8585
// Configs for pion
8686
struct : ConfigurableGroup {
87-
Configurable<bool> ITSPIDSelection{"ITSPIDSelection", true, "PID ITS"};
87+
Configurable<bool> itsPIDSelection{"itsPIDSelection", true, "PID ITS"};
8888
Configurable<float> lowITSPIDNsigma{"lowITSPIDNsigma", -3.0, "lower cut on PID nsigma for ITS"};
8989
Configurable<float> highITSPIDNsigma{"highITSPIDNsigma", 3.0, "higher cut on PID nsigma for ITS"};
90-
Configurable<int> ITSclusterPiMeson{"ITSclusterPiMeson", 5, "Minimum number of ITS cluster for pi meson track"};
91-
Configurable<int> TPCCrossedRowsPiMeson{"TPCCrossedRowsPiMeson", 80, "Minimum number of TPC Crossed Rows for pi meson track"};
92-
Configurable<float> CutDCAxyPiMeson{"CutDCAxyPiMeson", 0.1, "Maximum DCAxy for pi meson track"};
93-
Configurable<float> CutDCAzPiMeson{"CutDCAzPiMeson", 0.1, "Maximum DCAz for pi meson track"};
94-
Configurable<float> CutEtaPiMeson{"CutEtaPiMeson", 0.8, "Maximum eta for pi meson track"};
95-
Configurable<float> CutPTPiMeson{"CutPTPiMeson", 0.8, "Maximum pt for pi meson track"};
96-
Configurable<bool> UsePID{"UsePID", true, "Flag for using PID selection for pi meson track"};
90+
Configurable<int> itsclusterPiMeson{"itsclusterPiMeson", 5, "Minimum number of ITS cluster for pi meson track"};
91+
Configurable<int> tpcCrossedRowsPiMeson{"tpcCrossedRowsPiMeson", 80, "Minimum number of TPC Crossed Rows for pi meson track"};
92+
Configurable<float> cutDCAxyPiMeson{"cutDCAxyPiMeson", 0.1, "Maximum DCAxy for pi meson track"};
93+
Configurable<float> cutDCAzPiMeson{"cutDCAzPiMeson", 0.1, "Maximum DCAz for pi meson track"};
94+
Configurable<float> cutEtaPiMeson{"cutEtaPiMeson", 0.8, "Maximum eta for pi meson track"};
95+
Configurable<float> cutPTPiMeson{"cutPTPiMeson", 0.8, "Maximum pt for pi meson track"};
96+
Configurable<bool> usePID{"usePID", true, "Flag for using PID selection for pi meson track"};
9797
Configurable<float> nsigmaCutTPCPiMeson{"nsigmaCutTPCPiMeson", 3.0, "Maximum nsigma cut TPC for pi meson track"};
9898
Configurable<float> nsigmaCutTOFPiMeson{"nsigmaCutTOFPiMeson", 3.0, "Maximum nsigma cut TOF for pi meson track"};
99-
Configurable<float> CutTOFBetaPiMeson{"CutTOFBetaPiMeson", 3.0, "Maximum beta cut for pi meson track"};
99+
Configurable<float> cutTOFBetaPiMeson{"cutTOFBetaPiMeson", 3.0, "Maximum beta cut for pi meson track"};
100100
} grpPion;
101101

102102
// Configs for V0
@@ -110,7 +110,7 @@ struct cksspinalignment {
110110
Configurable<double> cMaxV0DCA{"cMaxV0DCA", 1.2, "Maximum V0 DCA to PV"};
111111
Configurable<float> cMinV0DCAPi{"cMinV0DCAPi", 0.05, "Minimum V0 daughters DCA to PV for Pi"};
112112
Configurable<float> cMaxV0LifeTime{"cMaxV0LifeTime", 50, "Maximum V0 life time"};
113-
Configurable<float> qtArmenterosMin{"qtArmenterosMinForK0", 0.2, "Minimum armenteros cut for K0s"};
113+
Configurable<float> qtArmenterosMin{"qtArmenterosMin", 0.2, "Minimum armenteros cut for K0s"};
114114
// config for V0 daughters
115115
Configurable<float> confDaughEta{"confDaughEta", 0.8f, "V0 Daugh sel: max eta"};
116116
Configurable<float> cfgDaughPiPt{"cfgDaughPiPt", 0.2, "minimum daughter pion pt"};
@@ -137,7 +137,7 @@ struct cksspinalignment {
137137
template <typename T>
138138
bool selectionTrack(const T& candidate)
139139
{
140-
if (candidate.isGlobalTrack() && candidate.isPVContributor() && candidate.itsNCls() >= grpPion.ITSclusterPiMeson && candidate.tpcNClsCrossedRows() > grpPion.TPCCrossedRowsPiMeson && std::abs(candidate.dcaXY()) <= grpPion.CutDCAxyPiMeson && std::abs(candidate.dcaZ()) <= grpPion.CutDCAzPiMeson && std::abs(candidate.eta()) <= grpPion.CutEtaPiMeson && candidate.pt() >= grpPion.CutPTPiMeson) {
140+
if (candidate.isGlobalTrack() && candidate.isPVContributor() && candidate.itsNCls() >= grpPion.itsclusterPiMeson && candidate.tpcNClsCrossedRows() > grpPion.tpcCrossedRowsPiMeson && std::abs(candidate.dcaXY()) <= grpPion.cutDCAxyPiMeson && std::abs(candidate.dcaZ()) <= grpPion.cutDCAzPiMeson && std::abs(candidate.eta()) <= grpPion.cutEtaPiMeson && candidate.pt() >= grpPion.cutPTPiMeson) {
141141
return true;
142142
}
143143
return false;
@@ -149,7 +149,7 @@ struct cksspinalignment {
149149
if (!candidate.hasTOF() && std::abs(candidate.tpcNSigmaPi()) < grpPion.nsigmaCutTPCPiMeson) {
150150
return true;
151151
}
152-
if (candidate.hasTOF() && candidate.beta() > grpPion.CutTOFBetaPiMeson && std::abs(candidate.tpcNSigmaPi()) < grpPion.nsigmaCutTPCPiMeson && std::abs(candidate.tofNSigmaPi()) < grpPion.nsigmaCutTOFPiMeson) {
152+
if (candidate.hasTOF() && candidate.beta() > grpPion.cutTOFBetaPiMeson && std::abs(candidate.tpcNSigmaPi()) < grpPion.nsigmaCutTPCPiMeson && std::abs(candidate.tofNSigmaPi()) < grpPion.nsigmaCutTOFPiMeson) {
153153
return true;
154154
}
155155
return false;
@@ -299,19 +299,19 @@ struct cksspinalignment {
299299
if (collision.triggereventep()) {
300300
histos.fill(HIST("hEvtSelInfo"), 2.5);
301301

302-
for (auto track1 : tracks) {
302+
for (const auto& track1 : tracks) {
303303
histos.fill(HIST("hTrkSelInfo"), 0.5);
304304
if (!selectionTrack(track1)) {
305305
continue;
306306
}
307307
histos.fill(HIST("hTrkSelInfo"), 1.5);
308308

309-
if (grpPion.ITSPIDSelection && !(itsResponse.nSigmaITS<o2::track::PID::Pion>(track1) > grpPion.lowITSPIDNsigma && itsResponse.nSigmaITS<o2::track::PID::Pion>(track1) < grpPion.highITSPIDNsigma)) {
309+
if (grpPion.itsPIDSelection && !(itsResponse.nSigmaITS<o2::track::PID::Pion>(track1) > grpPion.lowITSPIDNsigma && itsResponse.nSigmaITS<o2::track::PID::Pion>(track1) < grpPion.highITSPIDNsigma)) {
310310
continue;
311311
}
312312
histos.fill(HIST("hTrkSelInfo"), 2.5);
313313

314-
if (grpPion.UsePID && !selectionPID(track1)) {
314+
if (grpPion.usePID && !selectionPID(track1)) {
315315
continue;
316316
}
317317
histos.fill(HIST("hTrkSelInfo"), 3.5);

0 commit comments

Comments
 (0)