Skip to content

Commit 4589de3

Browse files
committed
Fix linter warnings
1 parent f1df02b commit 4589de3

File tree

4 files changed

+11
-9
lines changed

4 files changed

+11
-9
lines changed

PWGUD/Core/UPCHelpers.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@
2222
#include "CommonConstants/LHCConstants.h"
2323
#include "Framework/AnalysisDataModel.h"
2424

25-
#include "TLorentzVector.h"
26-
2725
using BCsWithBcSels = o2::soa::Join<o2::aod::BCs, o2::aod::BcSels>;
2826

2927
using ForwardTracks = o2::soa::Join<o2::aod::FwdTracks, o2::aod::FwdTracksCov>;
@@ -96,13 +94,15 @@ struct FITInfo {
9694
int32_t distClosestBcT0A = 999;
9795
};
9896

97+
constexpr double AbsorberMid = 26.5;
98+
9999
template <typename T, typename TSelectorsArray>
100100
void applyFwdCuts(UPCCutparHolder& upcCuts, const T& track, TSelectorsArray& fwdSelectors)
101101
{
102102
fwdSelectors[kFwdSelPt] = track.pt() > upcCuts.getFwdPtLow() && track.pt() < upcCuts.getFwdPtHigh(); // check pt
103103
fwdSelectors[kFwdSelEta] = track.eta() > upcCuts.getFwdEtaLow() && track.eta() < upcCuts.getFwdEtaHigh(); // check pseudorapidity
104104
fwdSelectors[kFwdSelRabs] = track.rAtAbsorberEnd() > upcCuts.getMuonRAtAbsorberEndLow() && track.rAtAbsorberEnd() < upcCuts.getMuonRAtAbsorberEndHigh(); // check muon R
105-
fwdSelectors[kFwdSelpDCA] = track.rAtAbsorberEnd() < 26.5 ? track.pDca() < upcCuts.getMuonPDcaHighFirst() : track.pDca() < upcCuts.getMuonPDcaHighSecond(); // check pDCA
105+
fwdSelectors[kFwdSelpDCA] = track.rAtAbsorberEnd() < AbsorberMid ? track.pDca() < upcCuts.getMuonPDcaHighFirst() : track.pDca() < upcCuts.getMuonPDcaHighSecond(); // check pDCA
106106
fwdSelectors[kFwdSelChi2] = track.chi2() > upcCuts.getFwdChi2Low() && track.chi2() < upcCuts.getFwdChi2High(); // check chi2
107107
}
108108

@@ -131,7 +131,7 @@ void applyBarrelCuts(UPCCutparHolder& upcCuts, const T& track, TSelectorsArray&
131131

132132
if (upcCuts.getCheckMaxDcaXY()) {
133133
float dca = track.dcaXY();
134-
float maxDCA = 0.0105f + 0.0350f / pow(track.pt(), 1.1f);
134+
float maxDCA = 0.0105f + 0.0350f / std::pow(track.pt(), 1.1f);
135135
barrelSelectors[kBarrelSelDCAXY] = dca < maxDCA;
136136
} else {
137137
barrelSelectors[kBarrelSelDCAXY] = true;

PWGUD/TableProducer/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ o2physics_add_dpl_workflow(two-tracks-event-table-producer
4242
COMPONENT_NAME Analysis)
4343

4444
o2physics_add_dpl_workflow(fwdtrack-propagation
45-
SOURCES fwdTrackPropagation.cxx
45+
SOURCES fwdtrackPropagation.cxx
4646
PUBLIC_LINK_LIBRARIES O2::Framework O2Physics::AnalysisCore O2::GlobalTracking
4747
COMPONENT_NAME Analysis)
4848

PWGUD/TableProducer/fwdTrackPropagation.cxx renamed to PWGUD/TableProducer/fwdtrackPropagation.cxx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,15 @@
2727
#include "Framework/runDataProcessing.h"
2828
#include "GlobalTracking/MatchGlobalFwd.h"
2929
#include "MCHTracking/TrackExtrap.h"
30-
#include "MCHTracking/TrackParam.h"
3130
#include "ReconstructionDataFormats/TrackFwd.h"
3231

3332
#include "Math/SMatrix.h"
3433
#include "TGeoGlobalMagField.h"
3534

35+
#include <map>
36+
#include <string>
37+
#include <vector>
38+
3639
using namespace o2::framework;
3740
using namespace o2::framework::expressions;
3841
using o2::aod::fwdtrack::ForwardTrackTypeEnum;

PWGUD/TableProducer/upcCandProducerMuon.cxx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616

1717
#include "PWGUD/Core/UPCCutparHolder.h"
1818
#include "PWGUD/Core/UPCHelpers.h"
19-
#include "PWGUD/DataModel/UDTables.h"
2019

2120
#include "CCDB/BasicCCDBManager.h"
2221
#include "CommonConstants/LHCConstants.h"
@@ -35,6 +34,7 @@
3534

3635
#include <algorithm>
3736
#include <map>
37+
#include <string>
3838
#include <vector>
3939

4040
using namespace o2::framework;
@@ -94,7 +94,6 @@ struct UpcCandProducerMuon {
9494

9595
bool cut(const o2::dataformats::GlobalFwdTrack& pft, const ForwardTracks::iterator& fwdTrack)
9696
{
97-
constexpr double AbsorberMid = 26.5;
9897
histRegistry.fill(HIST("MuonsSelCounter"), upchelpers::kFwdSelAll, 1);
9998
auto pt = pft.getPt();
10099
auto eta = pft.getEta();
@@ -104,7 +103,7 @@ struct UpcCandProducerMuon {
104103
bool passPt = pt > fUpcCuts.getFwdPtLow() && pt < fUpcCuts.getFwdPtHigh();
105104
bool passEta = eta > fUpcCuts.getFwdEtaLow() && eta < fUpcCuts.getFwdEtaHigh();
106105
bool passRabs = rabs > fUpcCuts.getMuonRAtAbsorberEndLow() && rabs < fUpcCuts.getMuonRAtAbsorberEndHigh();
107-
bool passPDca = rabs < AbsorberMid ? pdca < fUpcCuts.getMuonPDcaHighFirst() : pdca < fUpcCuts.getMuonPDcaHighSecond();
106+
bool passPDca = rabs < upchelpers::AbsorberMid ? pdca < fUpcCuts.getMuonPDcaHighFirst() : pdca < fUpcCuts.getMuonPDcaHighSecond();
108107
bool passChi2 = chi2 > fUpcCuts.getFwdChi2Low() && chi2 < fUpcCuts.getFwdChi2High();
109108
if (passPt)
110109
histRegistry.fill(HIST("MuonsSelCounter"), upchelpers::kFwdSelPt, 1);

0 commit comments

Comments
 (0)