Skip to content

Commit 02af08c

Browse files
committed
Fixes according to update of PR #7381
1 parent a9455e7 commit 02af08c

1 file changed

Lines changed: 35 additions & 37 deletions

File tree

PWGJE/Tasks/ChJetTriggerQATask.cxx

Lines changed: 35 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@
4141
#include "Framework/HistogramRegistry.h"
4242

4343
using namespace o2;
44-
using namespace framework;
45-
using namespace framework::expressions;
44+
using namespace o2::framework;
45+
using namespace o2::framework::expressions;
4646

4747
// What this task should do
4848
// Event by event fill
@@ -143,9 +143,9 @@ struct ChJetTriggerQATask {
143143
Filter jetRadiusSelection = (aod::jet::r == nround(cfgJetR.node() * 100.0f));
144144

145145
using filteredColl = soa::Filtered<soa::Join<JetCollisions, aod::JChTrigSels, aod::EvSels>>::iterator;
146-
using filteredJTracks = soa::Filtered<soa::Join<aod::JTracks, aod::JTrackPIs>>;
147-
// using Tracks = soa::Join<aod::Tracks, aod::TracksExtra>; // could be deleted? it is not iterator
146+
using filteredJTracks = soa::Filtered<soa::Join<aod::JTracks, aod::JTrackPIs, aod::JTrackExtras>>;
148147
using filteredJets = soa::Filtered<soa::Join<aod::ChargedJets, aod::ChargedJetConstituents>>;
148+
// using Tracks = soa::Join<aod::Tracks, aod::TracksExtra>; // could be deleted? it is not iterator
149149

150150
void process(filteredColl const& collision, filteredJTracks const& tracks, filteredJets const& jets /*, Tracks const&*/)
151151
{
@@ -182,66 +182,64 @@ struct ChJetTriggerQATask {
182182
spectra.fill(HIST("vertexZ"),
183183
collision.posZ()); // Inclusive Track Cross TPC Rows
184184

185-
for (auto const& trk : tracks) { // loop over filtered tracks in full TPC volume having pT > 100 MeV
185+
for (auto const& track : tracks) { // loop over filtered tracks in full TPC volume having pT > 100 MeV
186186

187-
auto const& originalTrack = trk.track_as<soa::Join<aod::Tracks, aod::TracksExtra, aod::TracksDCA>>();
187+
auto const& originalTrack = track.track_as<soa::Join<aod::Tracks, aod::TracksExtra>>();
188188

189-
spectra.fill(HIST("globalP_tpcglobalPDiff_withoutcuts"), trk.p(), trk.p() - originalTrack.tpcInnerParam());
189+
spectra.fill(HIST("globalP_tpcglobalPDiff_withoutcuts"), track.p(), track.p() - originalTrack.tpcInnerParam());
190190

191-
if (TMath::Abs(trk.phi() - TMath::Pi()) < phiAngleRestriction) {
192-
spectra.fill(HIST("globalP_tpcglobalPDiff_withoutcuts_phirestrict"), trk.p(), trk.p() - originalTrack.tpcInnerParam());
191+
if (TMath::Abs(track.phi() - TMath::Pi()) < phiAngleRestriction) {
192+
spectra.fill(HIST("globalP_tpcglobalPDiff_withoutcuts_phirestrict"), track.p(), track.p() - originalTrack.tpcInnerParam());
193193
}
194194

195-
if (!jetderiveddatautilities::selectTrack(trk, trackSelection)) {
195+
if (!jetderiveddatautilities::selectTrack(track, trackSelection)) {
196196
continue;
197197
}
198198

199199
if (originalTrack.itsNCls() >= 4) { // correspond to number of track hits in ITS layers
200-
auto dcaX = originalTrack.x() - collision.posX();
201-
auto dcaY = originalTrack.y() - collision.posY();
202-
spectra.fill(HIST("DCAx_track_Phi_pT"), dcaX, trk.phi(), trk.pt());
203-
spectra.fill(HIST("DCAy_track_Phi_pT"), dcaY, trk.phi(), trk.pt());
204-
spectra.fill(HIST("DCAz_track_Phi_pT"), originalTrack.dcaZ(), trk.phi(), trk.pt());
200+
spectra.fill(HIST("DCAx_track_Phi_pT"), track.dcaX(), track.phi(), track.pt());
201+
spectra.fill(HIST("DCAy_track_Phi_pT"), track.dcaY(), track.phi(), track.pt());
202+
spectra.fill(HIST("DCAz_track_Phi_pT"), track.dcaZ(), track.phi(), track.pt());
205203
}
206204

207-
spectra.fill(HIST("nITSClusters_TrackPt"), originalTrack.itsNCls(), trk.phi(), trk.pt());
205+
spectra.fill(HIST("nITSClusters_TrackPt"), originalTrack.itsNCls(), track.phi(), track.pt());
208206

209-
spectra.fill(HIST("globalP_tpcglobalPDiff"), trk.p(), trk.p() - originalTrack.tpcInnerParam());
210-
if (trk.p() > 0 && originalTrack.tpcInnerParam() > 0) {
211-
spectra.fill(HIST("global1overP_tpcglobalPDiff"), trk.p(), 1. / trk.p() - 1. / originalTrack.tpcInnerParam());
207+
spectra.fill(HIST("globalP_tpcglobalPDiff"), track.p(), track.p() - originalTrack.tpcInnerParam());
208+
if (track.p() > 0 && originalTrack.tpcInnerParam() > 0) {
209+
spectra.fill(HIST("global1overP_tpcglobalPDiff"), track.p(), 1. / track.p() - 1. / originalTrack.tpcInnerParam());
212210
}
213-
if (TMath::Abs(trk.phi() - TMath::Pi()) < phiAngleRestriction) {
214-
spectra.fill(HIST("globalP_tpcglobalPDiff_phirestrict"), trk.p(), trk.p() - originalTrack.tpcInnerParam());
211+
if (TMath::Abs(track.phi() - TMath::Pi()) < phiAngleRestriction) {
212+
spectra.fill(HIST("globalP_tpcglobalPDiff_phirestrict"), track.p(), track.p() - originalTrack.tpcInnerParam());
215213

216-
if (trk.p() > 0 && originalTrack.tpcInnerParam() > 0) {
217-
spectra.fill(HIST("global1overP_tpcglobalPDiff_phirestrict"), trk.p(), 1. / trk.p() - 1. / originalTrack.tpcInnerParam());
214+
if (track.p() > 0 && originalTrack.tpcInnerParam() > 0) {
215+
spectra.fill(HIST("global1overP_tpcglobalPDiff_phirestrict"), track.p(), 1. / track.p() - 1. / originalTrack.tpcInnerParam());
218216
}
219217
}
220218

221219
spectra.fill(
222-
HIST("ptphiTrackInclGood"), trk.pt(),
223-
trk.phi()); // Inclusive Track pT vs phi spectrum in TPC volume
220+
HIST("ptphiTrackInclGood"), track.pt(),
221+
track.phi()); // Inclusive Track pT vs phi spectrum in TPC volume
224222
spectra.fill(
225-
HIST("ptetaTrackInclGood"), trk.pt(),
226-
trk.eta()); // Inclusive Track pT vs eta spectrum in TPC volume
223+
HIST("ptetaTrackInclGood"), track.pt(),
224+
track.eta()); // Inclusive Track pT vs eta spectrum in TPC volume
227225

228226
if (bAddSupplementHistosToOutput) {
229227
spectra.fill(
230-
HIST("phietaTrackAllInclGood"), trk.eta(),
231-
trk.phi()); // Inclusive Track pT vs eta spectrum in TPC volume
228+
HIST("phietaTrackAllInclGood"), track.eta(),
229+
track.phi()); // Inclusive Track pT vs eta spectrum in TPC volume
232230

233-
if (trk.pt() > 5.0) {
231+
if (track.pt() > 5.0) {
234232
spectra.fill(
235-
HIST("phietaTrackHighPtInclGood"), trk.eta(),
236-
trk.phi()); // Inclusive Track pT vs eta spectrum in TPC volume
233+
HIST("phietaTrackHighPtInclGood"), track.eta(),
234+
track.phi()); // Inclusive Track pT vs eta spectrum in TPC volume
237235
}
238236
}
239237

240-
if (trk.pt() >
238+
if (track.pt() >
241239
leadingTrackPt) { // Find leading track pT in full TPC volume
242-
leadingTrackPt = trk.pt();
243-
leadingTrackEta = trk.eta();
244-
leadingTrackPhi = trk.phi();
240+
leadingTrackPt = track.pt();
241+
leadingTrackEta = track.eta();
242+
leadingTrackPhi = track.phi();
245243
}
246244
}
247245

@@ -322,4 +320,4 @@ WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
322320

323321
return WorkflowSpec{adaptAnalysisTask<ChJetTriggerQATask>(
324322
cfgc, TaskName{"jet-charged-trigger-qa"})};
325-
}
323+
}

0 commit comments

Comments
 (0)