Skip to content

Commit 2bb4ff3

Browse files
PWGLF: fix typos in configurable names (#1031)
1 parent 0ce73eb commit 2bb4ff3

1 file changed

Lines changed: 10 additions & 16 deletions

File tree

PWGLF/Tasks/spectraCharged.cxx

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -37,23 +37,18 @@ struct chargedSpectra {
3737
HistogramRegistry histos;
3838
Service<TDatabasePDG> pdg;
3939

40-
// TODO: can we derive this from config context resp from meta data in the file to avoid having to specify this option?
41-
Configurable<bool> isMC{"isMC", false, "option to flag mc"};
42-
Configurable<bool> isRun3{"isRun3", true, "Is Run3 dataset"}; // TODO: derive this from metadata once possible to get rid of the flag
43-
4440
// task settings that can be steered via hyperloop
45-
Configurable<uint32_t> maxMultMeas{"measMult", 100, "max measured multiplicity."};
46-
Configurable<uint32_t> maxMultTrue{"measTrue", 100, "max true multiplicity."};
47-
48-
// TODO: better use 2d configurables here
49-
Configurable<float> etaCut{"etaCut", 0.8f, "Eta cut."};
50-
Configurable<float> ptMinCut{"ptMinCut", 0.15f, "Pt min cut."};
51-
Configurable<float> ptMaxCut{"ptMaxCut", 10.f, "Pt max cut."};
41+
Configurable<bool> isRun3{"isRun3", true, "is Run3 dataset"}; // TODO: derive this from metadata once possible to get rid of the flag
42+
Configurable<uint32_t> maxMultMeas{"maxMultMeas", 100, "max measured multiplicity"};
43+
Configurable<uint32_t> maxMultTrue{"maxMultTrue", 100, "max true multiplicity"};
44+
Configurable<float> etaCut{"etaCut", 0.8f, "eta cut"};
45+
Configurable<float> ptMinCut{"ptMinCut", 0.15f, "pt min cut"};
46+
Configurable<float> ptMaxCut{"ptMaxCut", 10.f, "pt max cut"};
5247

5348
// helper struct to store transient properties
5449
struct varContainer {
55-
uint32_t multMeas{0};
56-
uint32_t multTrue{0};
50+
uint32_t multMeas{0u};
51+
uint32_t multTrue{0u};
5752
bool isAcceptedEvent{false};
5853
bool isAcceptedEventMC{false};
5954
bool isChargedPrimary{false};
@@ -138,7 +133,7 @@ void chargedSpectra::init(InitContext const&)
138133
histos.add("multDist_evt_meas", "", kTH1D, {multMeasAxis}); // measured event distribution (contains contamination from events not in specified class or with wrong vertex position)
139134
histos.add("multPtSpec_trk_meas", "", kTH2D, {multMeasAxis, ptMeasAxis}); // measured tracks (contains contamination from secondary particles, particles smeared into acceptance and tracks originating from background events as defined above )
140135

141-
if (isMC) {
136+
if (doprocessMC) {
142137

143138
const AxisSpec ptTrueAxis{ptBinEdges, "#it{p}_{T} (GeV/c)", "pt_true"};
144139

@@ -274,7 +269,7 @@ void chargedSpectra::initEvent(const C& collision, const T& tracks)
274269
vars.isAcceptedEvent = false;
275270
if (std::abs(collision.posZ()) < 10.f) {
276271
if (isRun3 ? collision.sel8() : collision.sel7()) {
277-
if ((isRun3 || isMC) ? true : collision.alias()[kINT7]) {
272+
if ((isRun3 || doprocessMC) ? true : collision.alias()[kINT7]) {
278273
vars.isAcceptedEvent = true;
279274
}
280275
}
@@ -296,7 +291,6 @@ void chargedSpectra::initEventMC(const C& collision, const P& particles)
296291
}
297292
++vars.multTrue;
298293
}
299-
// TODO: also determine event class and check if true z vtx positin is good
300294
vars.isAcceptedEventMC = ((std::abs(collision.posZ()) < 10.f) && (vars.multTrue > 0));
301295
}
302296

0 commit comments

Comments
 (0)