Skip to content

Commit 7af14ba

Browse files
authored
Fixing some o2linter warning/errors in the new task
1 parent ee873ab commit 7af14ba

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

PWGUD/Tasks/sgExclusiveJpsiMidrapidity.cxx

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ using namespace o2::framework::expressions;
3636
using LorentzVector = ROOT::Math::PxPyPzMVector;
3737

3838
// Struct to define the analysis task
39-
struct sgExclusiveJpsiMidrapidity {
39+
struct SgExclusiveJpsiMidrapidity {
4040
// SGSelector object to manage track and collision selections
4141
SGSelector sgSelector;
4242

@@ -89,10 +89,10 @@ struct sgExclusiveJpsiMidrapidity {
8989
// Fill counter to see effect of each selection criteria
9090
auto hSelectionCounter = registry.add<TH1>("hSelectionCounter", "hSelectionCounter;;NEvents", HistType::kTH1I, {{20, 0., 20.}});
9191

92-
TString SelectionCuts[16] = {"NoSelection", "gapside", "goodtracks", "truegap", "2collcontrib", "2goodtrk", "TPCPID", "rapCut", "unlikesign", "mass_cut", "coherent", "incoherent", "likesign", "mass_cut", "coherent", "incoherent"};
92+
numSelectionCuts] = {"NoSelection", "gapside", "goodtracks", "truegap", "2collcontrib", "2goodtrk", "TPCPID", "rapCut", "unlikesign", "mass_cut", "coherent", "incoherent", "likesign", "mass_cut", "coherent", "incoherent"};
9393

9494
for (int i = 0; i < numSelectionCuts; i++) {
95-
hSelectionCounter->GetXaxis()->SetBinLabel(i + 1, SelectionCuts[i].Data());
95+
hSelectionCounter->GetXaxis()->SetBinLabel(i + 1, selectionCuts[i].Data());
9696
}
9797
// tracks
9898
registry.add("hTracks", "N_{tracks}", kTH1F, {{100, -0.5, 99.5}});
@@ -131,18 +131,17 @@ struct sgExclusiveJpsiMidrapidity {
131131
registry.add("TwoPion/hPtLike", "Pt;#it{p_{t}}, GeV/c;", kTH1D, {{1000, 0., 10.}});
132132
registry.add("TwoPion/hEta", "Eta;#it{#eta};", kTH1F, {{500, -10., 10.}});
133133
registry.add("TwoPion/hRap", "Rapidity;#it{y};", kTH1F, {{500, -10., 10.}});
134-
registry.add("TwoPion/hPhiSystem", "Phi;#it{#Phi};", kTH1F, {{250, 0. * TMath::Pi(), 2. * TMath::Pi()}});
134+
registry.add("TwoPion/hPhiSystem", "Phi;#it{#Phi};", kTH1F, {{250, 0., o2::constants::math::TwoPI}});
135135
registry.add("TwoPion/hMPt", "Inv.M vs Pt;M, GeV/c^{2};#it{P_{t}}, GeV/c;", kTH2F, {{100, 0., 10.}, {100, 0., 10.}});
136136
}
137137

138-
using udtracks = soa::Join<aod::UDTracks, aod::UDTracksExtra, aod::UDTracksPID>;
139-
using udtracksfull = soa::Join<aod::UDTracks, aod::UDTracksPID, aod::UDTracksExtra, aod::UDTracksFlags, aod::UDTracksDCA>;
138+
using UDTracksFull = soa::Join<aod::UDTracks, aod::UDTracksPID, aod::UDTracksExtra, aod::UDTracksFlags, aod::UDTracksDCA>;
140139

141140
using UDCollisionsFull = soa::Join<aod::UDCollisions, aod::SGCollisions, aod::UDCollisionsSels, aod::UDZdcsReduced>;
142141

143142
//__________________________________________________________________________
144143
// Main process
145-
void process(UDCollisionsFull::iterator const& collision, udtracksfull const& tracks)
144+
void process(UDCollisionsFull::iterator const& collision, UDTracksFull const& tracks)
146145
{
147146
// No selection criteria
148147
registry.fill(HIST("hSelectionCounter"), 0);
@@ -155,8 +154,7 @@ struct sgExclusiveJpsiMidrapidity {
155154
registry.fill(HIST("hSelectionCounter"), 1);
156155

157156
// Accessing FIT information for further exclusivity and/or inclusivity
158-
float FIT_cut[5] = {fv0Cut, ft0aCut, ft0cCut, fddaCut, fddcCut};
159-
int truegapSide = sgSelector.trueGap(collision, FIT_cut[0], FIT_cut[1], FIT_cut[2], zdcCut);
157+
int truegapSide = sgSelector.trueGap(collision, fv0Cut, ft0aCut, ft0cCut, zdcCut);
160158

161159
// Initiating track parameters to select good tracks, values to be optimized in the configurables, parameters will be taken from SGtrackselector.h task included in the header
162160
std::vector<float> parameters = {pvCut, dcazCut, dcaxyCut, tpcChi2Cut, tpcNClsFindableCut, itsChi2Cut, etaCut, ptCut};
@@ -332,5 +330,5 @@ struct sgExclusiveJpsiMidrapidity {
332330
WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
333331
{
334332
return WorkflowSpec{
335-
adaptAnalysisTask<sgExclusiveJpsiMidrapidity>(cfgc)};
333+
adaptAnalysisTask<SgExclusiveJpsiMidrapidity>(cfgc)};
336334
}

0 commit comments

Comments
 (0)