Skip to content

Commit 5621318

Browse files
committed
Fix O2 linter issues
1 parent 80bbb34 commit 5621318

1 file changed

Lines changed: 16 additions & 16 deletions

File tree

‎PWGJE/Tasks/trackEfficiency.cxx‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ struct TrackEfficiency {
131131
}
132132
} else {
133133
const auto& aodTrack = jetTrack.template track_as<soa::Join<aod::Tracks, aod::TracksExtra, aod::TracksDCA>>();
134-
if (effSystMinNCrossedRowsTPCUseAlternateCut && (aodTrack.tpcNClsCrossedRows() < 120 - 5. / aodTrack.pt())) {
134+
if (effSystMinNCrossedRowsTPCUseAlternateCut && (aodTrack.tpcNClsCrossedRows() < 120 - 5. / aodTrack.pt())) { // o2-linter: disable=magic-number (TPC nCrossedRows calculation)
135135
return false;
136136
}
137137
if (customTrackSelection.IsSelected(aodTrack)) {
@@ -1634,11 +1634,11 @@ struct TrackEfficiency {
16341634
}
16351635

16361636
auto collTracks = jetTracks.sliceBy(tracksPerJCollision, collision.globalIndex());
1637-
int ntrack_nonassociatedtrack = 0;
1638-
int ntrack_associatedtrack_nonprimary = 0;
1639-
int ntrack_associatedtrack_primary = 0;
1640-
int ntrack_associatedtrack_split_nonprimary = 0;
1641-
int ntrack_associatedtrack_split_primary = 0;
1637+
int nTrackNonassociatedTrack = 0;
1638+
int nTrackAssociatedTrackNonprimary = 0;
1639+
int nTrackAssociatedTrackPrimary = 0;
1640+
int nTrackAssociatedTrackSplitNonprimary = 0;
1641+
int nTrackAssociatedTrackSplitPrimary = 0;
16421642
for (auto const& track : collTracks) {
16431643
registry.fill(HIST("hTrackCutsCounts"), 0.5);
16441644

@@ -1648,7 +1648,7 @@ struct TrackEfficiency {
16481648
registry.fill(HIST("hTrackCutsCounts"), 1.5);
16491649

16501650
if (!track.has_mcParticle()) {
1651-
ntrack_nonassociatedtrack += 1;
1651+
nTrackNonassociatedTrack += 1;
16521652

16531653
registry.fill(HIST("h3_track_pt_track_eta_track_phi_nonassociatedtrack"), track.pt(), track.eta(), track.phi());
16541654

@@ -1659,7 +1659,7 @@ struct TrackEfficiency {
16591659

16601660
auto jMcParticleFromTrack = track.mcParticle_as<JetParticlesWithOriginal>();
16611661
if (!jMcParticleFromTrack.isPhysicalPrimary()) {
1662-
ntrack_associatedtrack_nonprimary += 1;
1662+
nTrackAssociatedTrackNonprimary += 1;
16631663

16641664
registry.fill(HIST("h3_track_pt_track_eta_track_phi_associatedtrack_nonprimary"), track.pt(), track.eta(), track.phi());
16651665
registry.fill(HIST("h3_particle_pt_particle_eta_particle_phi_associatedtrack_nonprimary"), jMcParticleFromTrack.pt(), jMcParticleFromTrack.eta(), jMcParticleFromTrack.phi());
@@ -1668,7 +1668,7 @@ struct TrackEfficiency {
16681668
registry.fill(HIST("h3_particle_pt_high_particle_eta_particle_phi_associatedtrack_nonprimary"), jMcParticleFromTrack.pt(), jMcParticleFromTrack.eta(), jMcParticleFromTrack.phi());
16691669

16701670
if (std::find(seenMcParticlesVector.begin(), seenMcParticlesVector.end(), jMcParticleFromTrack.globalIndex()) != seenMcParticlesVector.end()) {
1671-
ntrack_associatedtrack_split_nonprimary += 1;
1671+
nTrackAssociatedTrackSplitNonprimary += 1;
16721672

16731673
registry.fill(HIST("h3_track_pt_track_eta_track_phi_associatedtrack_split_nonprimary"), track.pt(), track.eta(), track.phi());
16741674
registry.fill(HIST("h3_particle_pt_particle_eta_particle_phi_associatedtrack_split_nonprimary"), jMcParticleFromTrack.pt(), jMcParticleFromTrack.eta(), jMcParticleFromTrack.phi());
@@ -1684,7 +1684,7 @@ struct TrackEfficiency {
16841684

16851685
registry.fill(HIST("hTrackCutsCounts"), 3.5);
16861686

1687-
ntrack_associatedtrack_primary += 1;
1687+
nTrackAssociatedTrackPrimary += 1;
16881688
registry.fill(HIST("h3_track_pt_track_eta_track_phi_associatedtrack_primary"), track.pt(), track.eta(), track.phi());
16891689
registry.fill(HIST("h3_particle_pt_particle_eta_particle_phi_associatedtrack_primary"), jMcParticleFromTrack.pt(), jMcParticleFromTrack.eta(), jMcParticleFromTrack.phi());
16901690
registry.fill(HIST("h2_particle_pt_track_pt_residual_associatedtrack_primary"), jMcParticleFromTrack.pt(), (jMcParticleFromTrack.pt() - track.pt()) / jMcParticleFromTrack.pt());
@@ -1694,7 +1694,7 @@ struct TrackEfficiency {
16941694
registry.fill(HIST("h2_particle_pt_high_track_pt_high_residual_associatedtrack_primary"), jMcParticleFromTrack.pt(), (jMcParticleFromTrack.pt() - track.pt()) / jMcParticleFromTrack.pt());
16951695

16961696
if (std::find(seenMcParticlesVector.begin(), seenMcParticlesVector.end(), jMcParticleFromTrack.globalIndex()) != seenMcParticlesVector.end()) {
1697-
ntrack_associatedtrack_split_primary += 1;
1697+
nTrackAssociatedTrackSplitPrimary += 1;
16981698
registry.fill(HIST("h3_track_pt_track_eta_track_phi_associatedtrack_split_primary"), track.pt(), track.eta(), track.phi());
16991699
registry.fill(HIST("h3_particle_pt_particle_eta_particle_phi_associatedtrack_split_primary"), jMcParticleFromTrack.pt(), jMcParticleFromTrack.eta(), jMcParticleFromTrack.phi());
17001700

@@ -1708,11 +1708,11 @@ struct TrackEfficiency {
17081708
registry.fill(HIST("hTrackCutsCounts"), 4.5);
17091709
}
17101710
}
1711-
registry.fill(HIST("h_ntrack_nonassociatedtrack"), ntrack_nonassociatedtrack);
1712-
registry.fill(HIST("h_ntrack_associatedtrack_nonprimary"), ntrack_associatedtrack_nonprimary);
1713-
registry.fill(HIST("h_ntrack_associatedtrack_split_nonprimary"), ntrack_associatedtrack_split_nonprimary);
1714-
registry.fill(HIST("h_ntrack_associatedtrack_primary"), ntrack_associatedtrack_primary);
1715-
registry.fill(HIST("h_ntrack_associatedtrack_split_primary"), ntrack_associatedtrack_split_primary);
1711+
registry.fill(HIST("h_ntrack_nonassociatedtrack"), nTrackNonassociatedTrack);
1712+
registry.fill(HIST("h_ntrack_associatedtrack_nonprimary"), nTrackAssociatedTrackNonprimary);
1713+
registry.fill(HIST("h_ntrack_associatedtrack_split_nonprimary"), nTrackAssociatedTrackSplitNonprimary);
1714+
registry.fill(HIST("h_ntrack_associatedtrack_primary"), nTrackAssociatedTrackPrimary);
1715+
registry.fill(HIST("h_ntrack_associatedtrack_split_primary"), nTrackAssociatedTrackSplitPrimary);
17161716
}
17171717
}
17181718
PROCESS_SWITCH(TrackEfficiency, processQcCheck, "Histograms for QC checks", false);

0 commit comments

Comments
 (0)