Skip to content

[DPG] Please consider the following formatting changes to #10451 #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
alibuild wants to merge 1 commit into from
Closed

[DPG] Please consider the following formatting changes to #10451 #8

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion DPG/Tasks/TPC/tpcSkimsTableCreator.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,9 @@
Configurable<int> applyEvSel{"applyEvSel", 2, "Flag to apply rapidity cut: 0 -> no event selection, 1 -> Run 2 event selection, 2 -> Run 3 event selection"};
Configurable<int> trackSelection{"trackSelection", 1, "Track selection: 0 -> No Cut, 1 -> kGlobalTrack, 2 -> kGlobalTrackWoPtEta, 3 -> kGlobalTrackWoDCA, 4 -> kQualityTracks, 5 -> kInAcceptanceTracks"};
/// Configurables downsampling
Configurable<double> dwnSmplFactor_Pi{"dwnSmplFactor_Pi", 1., "downsampling factor for pions, default fraction to keep is 1."};

Check warning on line 63 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use lowerCamelCase for names of functions and variables.
Configurable<double> dwnSmplFactor_Pr{"dwnSmplFactor_Pr", 1., "downsampling factor for protons, default fraction to keep is 1."};

Check warning on line 64 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use lowerCamelCase for names of functions and variables.
Configurable<double> dwnSmplFactor_El{"dwnSmplFactor_El", 1., "downsampling factor for electrons, default fraction to keep is 1."};

Check warning on line 65 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use lowerCamelCase for names of functions and variables.
Configurable<float> sqrtSNN{"sqrt_s_NN", 0., "sqrt(s_NN), used for downsampling with the Tsallis distribution"};
Configurable<float> downsamplingTsalisPions{"downsamplingTsalisPions", -1., "Downsampling factor to reduce the number of pions"};
Configurable<float> downsamplingTsalisProtons{"downsamplingTsalisProtons", -1., "Downsampling factor to reduce the number of protons"};
Expand Down Expand Up @@ -258,7 +258,7 @@
rowTPCTree.reserve(tracks.size());

/// Loop over v0 candidates
for (auto v0 : v0s) {

Check warning on line 261 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use constant references for non-modified iterators in range-based for loops.
auto posTrack = v0.posTrack_as<soa::Filtered<Trks>>();
auto negTrack = v0.negTrack_as<soa::Filtered<Trks>>();
// gamma
Expand All @@ -282,7 +282,7 @@
// Lambda
if (static_cast<bool>(posTrack.pidbit() & (1 << 2)) && static_cast<bool>(negTrack.pidbit() & (1 << 2))) {
if (downsampleTsalisCharged(posTrack.pt(), downsamplingTsalisProtons, sqrtSNN, o2::track::pid_constants::sMasses[o2::track::PID::Proton], maxPt4dwnsmplTsalisProtons)) {
if (TMath::Abs(posTrack.tofNSigmaPr()) <= nSigmaTOFdautrack) {

Check warning on line 285 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Replace ROOT entities with equivalents from standard C++ or from O2.
fillSkimmedV0Table(v0, posTrack, collision, posTrack.tpcNSigmaPr(), posTrack.tofNSigmaPr(), posTrack.tpcExpSignalPr(posTrack.tpcSignal()), o2::track::PID::Proton, runnumber, dwnSmplFactor_Pr, hadronicRate);
}
}
Expand All @@ -296,7 +296,7 @@
fillSkimmedV0Table(v0, posTrack, collision, posTrack.tpcNSigmaPi(), posTrack.tofNSigmaPi(), posTrack.tpcExpSignalPi(posTrack.tpcSignal()), o2::track::PID::Pion, runnumber, dwnSmplFactor_Pi, hadronicRate);
}
if (downsampleTsalisCharged(negTrack.pt(), downsamplingTsalisProtons, sqrtSNN, o2::track::pid_constants::sMasses[o2::track::PID::Proton], maxPt4dwnsmplTsalisProtons)) {
if (TMath::Abs(negTrack.tofNSigmaPr()) <= nSigmaTOFdautrack) {

Check warning on line 299 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Replace ROOT entities with equivalents from standard C++ or from O2.
fillSkimmedV0Table(v0, negTrack, collision, negTrack.tpcNSigmaPr(), negTrack.tofNSigmaPr(), negTrack.tpcExpSignalPr(negTrack.tpcSignal()), o2::track::PID::Proton, runnumber, dwnSmplFactor_Pr, hadronicRate);
}
}
Expand Down Expand Up @@ -331,7 +331,7 @@

rowTPCTreeWithTrkQA.reserve(tracks.size());
/// Loop over v0 candidates
for (auto v0 : v0s) {

Check warning on line 334 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use constant references for non-modified iterators in range-based for loops.
auto posTrack = v0.posTrack_as<Trks>();
auto negTrack = v0.negTrack_as<Trks>();

Expand Down Expand Up @@ -375,7 +375,7 @@
// Lambda
if (static_cast<bool>(posTrack.pidbit() & (1 << 2)) && static_cast<bool>(negTrack.pidbit() & (1 << 2))) {
if (downsampleTsalisCharged(posTrack.pt(), downsamplingTsalisProtons, sqrtSNN, o2::track::pid_constants::sMasses[o2::track::PID::Proton], maxPt4dwnsmplTsalisProtons)) {
if (TMath::Abs(posTrack.tofNSigmaPr()) <= nSigmaTOFdautrack) {

Check warning on line 378 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Replace ROOT entities with equivalents from standard C++ or from O2.
fillSkimmedV0TableWithTrQA(v0, posTrack, posTrackQA, existPosTrkQA, collision, posTrack.tpcNSigmaPr(), posTrack.tofNSigmaPr(), posTrack.tpcExpSignalPr(posTrack.tpcSignal()), o2::track::PID::Proton, runnumber, dwnSmplFactor_Pr, hadronicRate);
}
}
Expand All @@ -389,7 +389,7 @@
fillSkimmedV0TableWithTrQA(v0, posTrack, posTrackQA, existPosTrkQA, collision, posTrack.tpcNSigmaPi(), posTrack.tofNSigmaPi(), posTrack.tpcExpSignalPi(posTrack.tpcSignal()), o2::track::PID::Pion, runnumber, dwnSmplFactor_Pi, hadronicRate);
}
if (downsampleTsalisCharged(negTrack.pt(), downsamplingTsalisProtons, sqrtSNN, o2::track::pid_constants::sMasses[o2::track::PID::Proton], maxPt4dwnsmplTsalisProtons)) {
if (TMath::Abs(negTrack.tofNSigmaPr()) <= nSigmaTOFdautrack) {

Check warning on line 392 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Replace ROOT entities with equivalents from standard C++ or from O2.
fillSkimmedV0TableWithTrQA(v0, negTrack, negTrackQA, existNegTrkQA, collision, negTrack.tpcNSigmaPr(), negTrack.tofNSigmaPr(), negTrack.tpcExpSignalPr(negTrack.tpcSignal()), o2::track::PID::Proton, runnumber, dwnSmplFactor_Pr, hadronicRate);
}
}
Expand Down Expand Up @@ -479,7 +479,7 @@
double n = a + b / sqrts;
double T = c + d / sqrts;
double p0 = n * T;
double result = pow((1. + mt / p0), -n);

Check warning on line 482 in DPG/Tasks/TPC/tpcSkimsTableCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter 

Use std:: prefix for names from the std namespace.
return result;
};

Expand Down Expand Up @@ -678,7 +678,7 @@
((trackSelection == 2) && trk.isGlobalTrackWoPtEta()) ||
((trackSelection == 3) && trk.isGlobalTrackWoDCA()) ||
((trackSelection == 4) && trk.isQualityTrack()) ||
((trackSelection == 5) && trk.isInAcceptanceTrack()))){
((trackSelection == 5) && trk.isInAcceptanceTrack()))) {
continue;
}
// get the corresponding trackQA using labelTracks2TracKQA and get variables of interest
Expand Down
Loading