Skip to content

Commit bda56b7

Browse files
vkuceraalibuildvictor-gonzalez
authored
[PWGCF] Fix compilation warnings (AliceO2Group#8381)
Co-authored-by: ALICE Builder <[email protected]> Co-authored-by: Victor Gonzalez <[email protected]>
1 parent 4304310 commit bda56b7

File tree

13 files changed

+48
-52
lines changed

13 files changed

+48
-52
lines changed

‎PWGCF/EbyEFluctuations/Tasks/RobustFluctuationObservables.cxx‎

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@
1414

1515
#include <iostream>
1616
#include <string>
17+
#include <memory>
18+
#include <vector>
19+
#include <map>
1720

1821
#include "TF1.h"
1922
#include "TGraphErrors.h"
@@ -82,7 +85,7 @@ struct RobustFluctuationObservables {
8285
// for vertex vs time:
8386
bool flagShowInfo = false;
8487
int lastRunNumber = -1;
85-
int nBCsPerOrbit = 3564;
88+
uint64_t nBCsPerOrbit = 3564;
8689

8790
// bc position correlations
8891
int64_t prevOrbit = -1;
@@ -101,7 +104,7 @@ struct RobustFluctuationObservables {
101104
int64_t orbitSOR = -1;
102105
// int64_t bcSORbis = -1; // global bc of the start of the first orbit - try alternative
103106
int64_t nBCsPerTF = 1; // 128*3564; // duration of TF in bcs
104-
int64_t TFid = -1; // count time frames in a given run
107+
uint64_t TFid = 0; // count time frames in a given run
105108
bool flagWaitForNewTF = false;
106109
uint32_t nOrbitsPerTF = 0;
107110

@@ -154,7 +157,7 @@ struct RobustFluctuationObservables {
154157
// hand-made ITS ROF cut
155158
Configurable<int> nITSROF{"nITSROF", 6, "nITSROF"};
156159
Configurable<int> nITSROF_BC_offset{"nITSROF_BC_offset", 65, "nITSROF_BC_offset"};
157-
Configurable<int> nITSROF_BC_cutWidth{"nITSROF_BC_cutWidth", 40, "nITSROF_BC_cutWidth"};
160+
Configurable<uint64_t> nITSROF_BC_cutWidth{"nITSROF_BC_cutWidth", 40, "nITSROF_BC_cutWidth"};
158161
// Configurable<int> nITSROF_middle_cut_forITSonlyVert{"nITSROF_middle_cut_forITSonlyVert", 198/2 /*ROF=198 in pp*/, "nITSROF_middle_cut_forITSonlyVert"};
159162
// Configurable<int> nNoITSonlyVertices{"nNoITSonlyVertices", false, "nITSROF_middle_cut_forITSonlyVert"};
160163

@@ -163,7 +166,7 @@ struct RobustFluctuationObservables {
163166
Configurable<float> cutVzTrackT0diffUpper{"cutVzTrackT0diffUpper", 1., "cutVzTrackT0diffUpper, cm"};
164167

165168
// splitting of the orbit into several BC ranges
166-
Configurable<std::vector<int>> vSplitBCpointsOfTheOrbit{"SplitBCpointsOfTheOrbit", {1200, 2000, 3000}, "BC split points of the orbit"};
169+
Configurable<std::vector<uint64_t>> vSplitBCpointsOfTheOrbit{"SplitBCpointsOfTheOrbit", {1200, 2000, 3000}, "BC split points of the orbit"};
167170

168171
// orbit QA
169172
uint32_t orbitAtCollIndexZero = 0;
@@ -968,7 +971,7 @@ struct RobustFluctuationObservables {
968971
}
969972

970973
if (myDF_ID >= 0 && myDF_ID < nHistQAplotsDF) {
971-
int diffOrbits = (int32_t)orbit - (int32_t)orbitAtCollIndexZero;
974+
int diffOrbits = static_cast<int32_t>(orbit) - static_cast<int32_t>(orbitAtCollIndexZero);
972975
TString strDF = Form("DF_%d", static_cast<int>(DF_ID_raw));
973976
fV_h1D_Orbit_vs_CollIndex[myDF_ID]->Fill(collision.index(), diffOrbits);
974977
fV_h1D_Orbit_vs_CollIndex[myDF_ID]->SetTitle(strDF);
@@ -1474,23 +1477,23 @@ struct RobustFluctuationObservables {
14741477
// ##### check how often we analyze collision in the same BC (and also the vZ difference)
14751478
if (collision.selection_bit(o2::aod::evsel::kNoTimeFrameBorder)) {
14761479
if (prevBC != 9999) {
1477-
int32_t diff = (int32_t)collBC - (int32_t)prevBC;
1480+
int32_t diff = static_cast<int32_t>(collBC) - static_cast<int32_t>(prevBC);
14781481
histosEvent.fill(HIST("hBC_DIFF_to_previous"), diff);
14791482
if (diff == 0)
14801483
histosEvent.fill(HIST("hBC_DIFF_to_previous_vZvZ_2D"), vZ, prev_vZ);
14811484
}
14821485
if (prevBcInTF >= 0) {
1483-
int32_t diffBcInTF = (int32_t)bcInTF - (int32_t)prevBcInTF;
1486+
int32_t diffBcInTF = static_cast<int32_t>(bcInTF) - static_cast<int32_t>(prevBcInTF);
14841487
histosEvent.fill(HIST("hBCinTF_DIFF_to_previous"), diffBcInTF);
14851488
}
14861489
if (prevFoundBcInTF >= 0) {
1487-
int32_t diffGlobalBcInTF = (int32_t)foundBcInTF - (int32_t)prevFoundBcInTF;
1490+
int32_t diffGlobalBcInTF = static_cast<int32_t>(foundBcInTF) - static_cast<int32_t>(prevFoundBcInTF);
14881491
histosEvent.fill(HIST("hBCinTF_DIFF_to_previous_FOUND_BC"), diffGlobalBcInTF);
14891492
}
14901493

14911494
// global found BC:
14921495
if (prevGlobalFoundBC != 9999) {
1493-
int32_t diff = (int32_t)globalFoundBC - (int32_t)prevGlobalFoundBC;
1496+
int32_t diff = static_cast<int32_t>(globalFoundBC) - static_cast<int32_t>(prevGlobalFoundBC);
14941497
histosEvent.fill(HIST("hBC_DIFF_to_previous_FOUND_BC"), diff);
14951498

14961499
if (counterPVcontributorsAfterTPCcuts > 0) {
@@ -1620,8 +1623,8 @@ struct RobustFluctuationObservables {
16201623

16211624
histosEventBcInTF.fill(HIST("hGlobalTracks_vs_bcInTF"), bcInTF, nTracksGlobalAccepted);
16221625
}
1623-
histosEvent.fill(HIST("hOrbitStartFromCollIndexZeroAft"), (int32_t)orbit - (int32_t)orbitAtCollIndexZero);
1624-
histosEvent.fill(HIST("h2D_Orbit_vs_CollIndex_Aft"), collision.index(), (int32_t)orbit - (int32_t)orbitAtCollIndexZero);
1626+
histosEvent.fill(HIST("hOrbitStartFromCollIndexZeroAft"), static_cast<int32_t>(orbit) - static_cast<int32_t>(orbitAtCollIndexZero));
1627+
histosEvent.fill(HIST("h2D_Orbit_vs_CollIndex_Aft"), collision.index(), static_cast<int32_t>(orbit) - static_cast<int32_t>(orbitAtCollIndexZero));
16251628

16261629
histosEvent.fill(HIST("hMF"), magneticField);
16271630
int MFsign = magneticField > 0 ? +1 : -1;
@@ -1680,11 +1683,11 @@ struct RobustFluctuationObservables {
16801683
histosEvent.fill(HIST("hBCFound_Aft"), globalFoundBC);
16811684
histosEvent.fill(HIST("h2D_numContrib_vs_BC"), collBC, collision.numContrib());
16821685

1683-
int64_t diffFoundBC_vs_BC = (int64_t)globalFoundBC - (int64_t)collBC;
1686+
int64_t diffFoundBC_vs_BC = static_cast<int64_t>(globalFoundBC) - static_cast<int64_t>(collBC);
16841687
histosEvent.fill(HIST("h2D_diffFoundBC_vs_BC"), collBC, diffFoundBC_vs_BC);
16851688

16861689
if (collision.has_foundBC())
1687-
histosEvent.fill(HIST("h2D_diffFoundBC_vs_BC_inTF"), collBC, (int64_t)foundBcInTF - (int64_t)bcInTF);
1690+
histosEvent.fill(HIST("h2D_diffFoundBC_vs_BC_inTF"), collBC, static_cast<int64_t>(foundBcInTF) - static_cast<int64_t>(bcInTF));
16881691

16891692
// with FT0 conditions
16901693
if (isFT0) {

‎PWGCF/FemtoUniverse/Core/FemtoUniverseMath.h‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ class FemtoUniverseMath
7676
const ROOT::Math::PtEtaPhiMVector vecpart1(part1.pt(), part1.eta(), part1.phi(), mass1);
7777
const ROOT::Math::PtEtaPhiMVector vecpart2(part2.pt(), part2.eta(), part2.phi(), mass2);
7878
const ROOT::Math::PtEtaPhiMVector trackSum = vecpart1 + vecpart2;
79-
const ROOT::Math::PtEtaPhiMVector trackDiff = vecpart1 - vecpart2;
8079

8180
const float beta = trackSum.Beta();
8281
const float betax = beta * std::cos(trackSum.Phi()) * std::sin(trackSum.Theta());

‎PWGCF/FemtoUniverse/TableProducer/femtoUniverseProducerTask.cxx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1445,7 +1445,7 @@ struct femtoUniverseProducerTask {
14451445
std::vector<int> tmpPDGCodes = ConfMCTruthPDGCodes; // necessary due to some features of the Configurable
14461446
for (uint32_t pdg : tmpPDGCodes) {
14471447
if (static_cast<int>(pdg) == static_cast<int>(pdgCode)) {
1448-
if ((pdgCode == 333)) { // && (recoMcIds && recoMcIds->get().contains(particle.globalIndex()))) { // ATTENTION: all Phi mesons are NOT primary particles
1448+
if (pdgCode == 333) { // && (recoMcIds && recoMcIds->get().contains(particle.globalIndex()))) { // ATTENTION: all Phi mesons are NOT primary particles
14491449
pass = true;
14501450
} else {
14511451
if (particle.isPhysicalPrimary() || (ConfActivateSecondaries && recoMcIds && recoMcIds->get().contains(particle.globalIndex())))
@@ -1497,11 +1497,11 @@ struct femtoUniverseProducerTask {
14971497
if constexpr (resolveDaughs) {
14981498
childIDs[0] = 0;
14991499
childIDs[1] = 0;
1500-
for (int i = 0; i < tmpIDtrack.size(); i++) {
1500+
for (std::size_t i = 0; i < tmpIDtrack.size(); i++) {
15011501
const auto& particle = tracks.iteratorAt(tmpIDtrack[i] - tracks.begin().globalIndex());
15021502
for (int daughIndex = 0, n = std::min(2ul, particle.daughtersIds().size()); daughIndex < n; daughIndex++) {
15031503
// loop to find the corresponding index of the daughters
1504-
for (int j = 0; j < tmpIDtrack.size(); j++) {
1504+
for (std::size_t j = 0; j < tmpIDtrack.size(); j++) {
15051505
if (tmpIDtrack[j] == particle.daughtersIds()[daughIndex]) {
15061506
childIDs[daughIndex] = i - j;
15071507
break;

‎PWGCF/FemtoUniverse/Tasks/femtoUniverseEfficiencyBase.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,15 +283,15 @@ struct femtoUniverseEfficiencyBase {
283283
{
284284
/// Histogramming same event
285285
for (auto& part : grouppartsOneMCGen) {
286-
if (!ConfNoPDGPartOne && part.pidcut() != ConfPDGCodePartOne) {
286+
if (!ConfNoPDGPartOne && part.tempFitVar() != ConfPDGCodePartOne) {
287287
continue;
288288
}
289289
trackHistoPartOneGen.fillQA<isMC, false>(part);
290290
}
291291

292292
if (!ConfIsSame) {
293293
for (auto& part : grouppartsTwoMCGen) {
294-
if (!ConfNoPDGPartTwo && part.pidcut() != ConfPDGCodePartTwo) {
294+
if (!ConfNoPDGPartTwo && part.tempFitVar() != ConfPDGCodePartTwo) {
295295
continue;
296296
}
297297
trackHistoPartTwoGen.fillQA<isMC, false>(part);

‎PWGCF/FemtoUniverse/Tasks/femtoUniversePairTaskTrackCascadeExtended.cxx‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ struct femtoUniversePairTaskTrackCascadeExtended {
263263

264264
void processSameEvent(FilteredFDCollision& col, FemtoFullParticles& parts)
265265
{
266-
const auto& magFieldTesla = col.magField();
266+
// const auto& magFieldTesla = col.magField();
267267

268268
auto groupPartsOne = partsOne->sliceByCached(aod::femtouniverseparticle::fdCollisionId, col.globalIndex(), cache);
269269
auto groupPartsTwo = partsTwo->sliceByCached(aod::femtouniverseparticle::fdCollisionId, col.globalIndex(), cache);
@@ -307,9 +307,9 @@ struct femtoUniversePairTaskTrackCascadeExtended {
307307
continue;
308308
if (!IsParticleCombined(p1, ConfTrackChoicePartOne))
309309
continue;
310-
const auto& posChild = parts.iteratorAt(p2.index() - 3);
311-
const auto& negChild = parts.iteratorAt(p2.index() - 2);
312-
const auto& bachelor = parts.iteratorAt(p2.index() - 1);
310+
// const auto& posChild = parts.iteratorAt(p2.index() - 3);
311+
// const auto& negChild = parts.iteratorAt(p2.index() - 2);
312+
// const auto& bachelor = parts.iteratorAt(p2.index() - 1);
313313

314314
/// Child particles must pass this condition to be selected
315315
/*if (!IsParticleTPC(posChild, CascChildTable[ConfCascType1][0]) || !IsParticleTPC(negChild, CascChildTable[ConfCascType1][1]) || !IsParticleTPC(bachelor, CascChildTable[ConfCascType1][2]))
@@ -320,7 +320,7 @@ struct femtoUniversePairTaskTrackCascadeExtended {
320320
}
321321
PROCESS_SWITCH(femtoUniversePairTaskTrackCascadeExtended, processSameEvent, "Enable processing same event for track - cascade", true);
322322

323-
void processMixedEvent(FilteredFDCollisions& cols, FemtoFullParticles& parts)
323+
void processMixedEvent(FilteredFDCollisions& cols, FemtoFullParticles& /*parts*/)
324324
{
325325
ColumnBinningPolicy<aod::collision::PosZ, aod::femtouniversecollision::MultNtr> colBinning{{ConfVtxBins, ConfMultBins}, true};
326326

@@ -341,9 +341,9 @@ struct femtoUniversePairTaskTrackCascadeExtended {
341341
if (!IsParticleCombined(p1, ConfTrackChoicePartOne))
342342
continue;
343343

344-
const auto& posChild = parts.iteratorAt(p2.index() - 3);
345-
const auto& negChild = parts.iteratorAt(p2.index() - 2);
346-
const auto& bachelor = parts.iteratorAt(p2.index() - 1);
344+
// const auto& posChild = parts.iteratorAt(p2.index() - 3);
345+
// const auto& negChild = parts.iteratorAt(p2.index() - 2);
346+
// const auto& bachelor = parts.iteratorAt(p2.index() - 1);
347347

348348
/// Child particles must pass this condition to be selected
349349
/*if (!IsParticleTPC(posChild, CascChildTable[ConfCascType1][0]) || !IsParticleTPC(negChild, CascChildTable[ConfCascType1][1]) || !IsParticleTPC(bachelor, CascChildTable[ConfCascType1][2]))

‎PWGCF/FemtoUniverse/Tasks/femtoUniversePairTaskTrackPhi.cxx‎

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
/// \author Zuzanna Chochulska, WUT Warsaw & CTU Prague, [email protected]
1818

1919
#include <vector>
20+
#include <string>
2021
#include "Framework/AnalysisTask.h"
2122
#include "Framework/runDataProcessing.h"
2223
#include "Framework/HistogramRegistry.h"
@@ -72,8 +73,8 @@ struct femtoUniversePairTaskTrackPhi {
7273
struct : o2::framework::ConfigurableGroup {
7374
Configurable<std::string> ConfEfficiencyTrackPath{"ConfEfficiencyTrackPath", "", "Local path to proton efficiency TH2F file"};
7475
Configurable<std::string> ConfEfficiencyPhiPath{"ConfEfficiencyPhiPath", "", "Local path to Phi efficiency TH2F file"};
75-
Configurable<long> ConfEfficiencyTrackTimestamp{"ConfEfficiencyTrackTimestamp", 0, "(long int) Timestamp for hadron"};
76-
Configurable<long> ConfEfficiencyPhiTimestamp{"ConfEfficiencyPhiTimestamp", 0, "(long int) Timestamp for phi"};
76+
Configurable<int64_t> ConfEfficiencyTrackTimestamp{"ConfEfficiencyTrackTimestamp", 0, "(int64_t) Timestamp for hadron"};
77+
Configurable<int64_t> ConfEfficiencyPhiTimestamp{"ConfEfficiencyPhiTimestamp", 0, "(int64_t) Timestamp for phi"};
7778
} ConfEff;
7879

7980
struct : o2::framework::ConfigurableGroup {
@@ -438,7 +439,7 @@ struct femtoUniversePairTaskTrackPhi {
438439
ccdb->setCaching(true);
439440
ccdb->setLocalObjectValidityChecking();
440441

441-
long now = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::system_clock::now().time_since_epoch()).count();
442+
int64_t now = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::system_clock::now().time_since_epoch()).count();
442443
ccdb->setCreatedNotAfter(now);
443444

444445
if (!ConfEff.ConfEfficiencyTrackPath.value.empty()) {
@@ -584,10 +585,11 @@ struct femtoUniversePairTaskTrackPhi {
584585
float mMassTwo = TDatabasePDG::Instance()->GetParticle(-321)->Mass(); // FIXME: Get from the PDG service of the common header
585586

586587
for (auto& [kaon1, kaon2] : combinations(CombinationsStrictlyUpperIndexPolicy(groupPartsKaons, groupPartsKaons))) {
587-
if (!IsKaonNSigma(kaon1.p(), trackCuts.getNsigmaTPC(kaon1, o2::track::PID::Kaon), trackCuts.getNsigmaTOF(kaon1, o2::track::PID::Kaon))) {
588-
}
589-
if (!IsKaonNSigma(kaon2.p(), trackCuts.getNsigmaTPC(kaon2, o2::track::PID::Kaon), trackCuts.getNsigmaTOF(kaon2, o2::track::PID::Kaon))) {
590-
}
588+
// empty if statements commented out on 20241114 to get rid of MegaLinter errors
589+
// if (!IsKaonNSigma(kaon1.p(), trackCuts.getNsigmaTPC(kaon1, o2::track::PID::Kaon), trackCuts.getNsigmaTOF(kaon1, o2::track::PID::Kaon))) {
590+
// }
591+
// if (!IsKaonNSigma(kaon2.p(), trackCuts.getNsigmaTPC(kaon2, o2::track::PID::Kaon), trackCuts.getNsigmaTOF(kaon2, o2::track::PID::Kaon))) {
592+
// }
591593
if ((kaon1.mAntiLambda() == 1) && (kaon2.mAntiLambda() == 1)) {
592594
part1Vec.SetPtEtaPhiM(kaon1.pt(), kaon1.eta(), kaon1.phi(), mMassOne);
593595
part2Vec.SetPtEtaPhiM(kaon2.pt(), kaon2.eta(), kaon2.phi(), mMassOne);
@@ -765,7 +767,7 @@ struct femtoUniversePairTaskTrackPhi {
765767
registryMCtruth.fill(HIST("MCtruthKp"), part.pt(), part.eta());
766768
registryMCtruth.fill(HIST("MCtruthKpPt"), part.pt());
767769
}
768-
if ((pdgCode == 333)) {
770+
if (pdgCode == 333) {
769771
registryMCtruth.fill(HIST("MCtruthPhi"), part.pt(), part.eta());
770772
continue;
771773
}

‎PWGCF/FemtoUniverse/Tasks/femtoUniversePairTaskTrackV0Extended.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ struct femtoUniversePairTaskTrackV0Extended {
517517
PROCESS_SWITCH(femtoUniversePairTaskTrackV0Extended, processMCSameEvent, "Enable processing same event for MC truth track - V0", false);
518518

519519
/// This function processes MC same events for V0 - V0
520-
void processMCSameEventV0(FilteredFDCollision& col, FemtoFullParticles& parts)
520+
void processMCSameEventV0(FilteredFDCollision& col, FemtoFullParticles& /*parts*/)
521521
{
522522
auto groupPartsTwo = partsTwoMC->sliceByCached(aod::femtouniverseparticle::fdCollisionId, col.globalIndex(), cache);
523523
const int multCol = ConfUseCent ? col.multV0M() : col.multNtr();
@@ -753,7 +753,7 @@ struct femtoUniversePairTaskTrackV0Extended {
753753
PROCESS_SWITCH(femtoUniversePairTaskTrackV0Extended, processMCMixedEvent, "Enable processing mixed events for MC truth track - V0", false);
754754

755755
/// This function processes MC mixed events for V0 - V0
756-
void processMCMixedEventV0(FilteredFDCollisions& cols, FemtoFullParticles& parts)
756+
void processMCMixedEventV0(FilteredFDCollisions& cols, FemtoFullParticles& /*parts*/)
757757
{
758758
ColumnBinningPolicy<aod::collision::PosZ, aod::femtouniversecollision::MultNtr> colBinningMult{{ConfVtxBins, ConfMultBins}, true};
759759
ColumnBinningPolicy<aod::collision::PosZ, aod::femtouniversecollision::MultV0M> colBinningCent{{ConfVtxBins, ConfMultBins}, true};

‎PWGCF/Flow/TableProducer/ZDCQvectors.cxx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,7 @@ struct ZDCqvectors {
365365
cal.calibList[iteration][step] = ccdb->getForTimeStamp<TList>(ccdb_dir, timestamp);
366366

367367
if (cal.calibList[iteration][step]) {
368-
for (int i = 0; i < names.size(); i++) {
368+
for (std::size_t i = 0; i < names.size(); i++) {
369369
TObject* obj = reinterpret_cast<TObject*>(cal.calibList[iteration][step]->FindObject(Form("%s", names[i].Data())));
370370
if (!obj) {
371371
if (counter < 1) {
@@ -426,7 +426,7 @@ struct ZDCqvectors {
426426
double getCorrection(int iteration, int step, const char* objName)
427427
{
428428
T* hist = nullptr;
429-
double calibConstant;
429+
double calibConstant{0};
430430

431431
hist = reinterpret_cast<T*>(cal.calibList[iteration][step]->FindObject(Form("%s", objName)));
432432
if (!hist) {
@@ -460,7 +460,7 @@ struct ZDCqvectors {
460460
sparsePars.push_back(h->GetAxis(3)->FindBin(v[1]));
461461
sparsePars.push_back(h->GetAxis(4)->FindBin(v[2]));
462462
}
463-
for (int i = 0; i < sparsePars.size(); i++) {
463+
for (std::size_t i = 0; i < sparsePars.size(); i++) {
464464
h->GetAxis(i)->SetRange(sparsePars[i], sparsePars[i]);
465465
}
466466
calibConstant = h->Projection(sparsePars.size())->GetMean();

‎PWGCF/Flow/Tasks/FlowGFWPbPb.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ struct FlowGFWPbPb {
162162
ccdb->setCreatedNotAfter(nolaterthan.value);
163163

164164
// Add some output objects to the histogram registry
165-
registry.add("hEventCount", "Number of Events;; No. of Events", {HistType::kTH1D, {{kNOOFEVENTSTEPS, -0.5, kNOOFEVENTSTEPS - 0.5}}});
165+
registry.add("hEventCount", "Number of Events;; No. of Events", {HistType::kTH1D, {{kNOOFEVENTSTEPS, -0.5, +kNOOFEVENTSTEPS - 0.5}}});
166166
registry.get<TH1>(HIST("hEventCount"))->GetXaxis()->SetBinLabel(kFILTERED + 1, "Filtered events");
167167
registry.get<TH1>(HIST("hEventCount"))->GetXaxis()->SetBinLabel(kSEL8 + 1, "Sel8");
168168
registry.get<TH1>(HIST("hEventCount"))->GetXaxis()->SetBinLabel(kOCCUPANCY + 1, "Occupancy");

‎PWGCF/Flow/Tasks/FlowRunbyRun.cxx‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,6 @@ struct FlowRunbyRun {
185185
for (auto& track : tracks) {
186186
TH1sList[runNumber][hPhi]->Fill(track.phi());
187187
TH1sList[runNumber][hEta]->Fill(track.eta());
188-
bool WithinPtPOI = (cfgCutPtPOIMin < track.pt()) && (track.pt() < cfgCutPtPOIMax); // within POI pT range
189188
bool WithinPtRef = (cfgCutPtRefMin < track.pt()) && (track.pt() < cfgCutPtRefMax); // within RF pT range
190189
if (WithinPtRef) {
191190
fGFW->Fill(track.eta(), 1, track.phi(), wacc * weff, 1);

0 commit comments

Comments
 (0)