[PWGLF] Modification in event selection cuts#11805
[PWGLF] Modification in event selection cuts#11805abmodak merged 8 commits intoAliceO2Group:masterfrom
Conversation
tutripat
requested review from
abmodak,
alibuild,
gbencedi and
skundu692
as code owners
|
O2 linter results: ❌ 33 errors, |
github-actions
bot
changed the title
| ((phi > ((M_PI / 2. - 0.1) * M_PI) - cfgPhiCut) && | ||
| (phi < ((M_PI / 2. - 0.1) * M_PI) + cfgPhiCut))) | ||
| ((phi > o2::constants::math::PI - cfgPhiCut) && (phi < o2::constants::math::PI + cfgPhiCut)) || | ||
| (phi > 2. * o2::constants::math::PI - cfgPhiCut) || |
There was a problem hiding this comment.
Please replace 2. * o2::constants::math::PI by o2::constants::math::TwoPI
| AxisSpec DeltaZAxis = {61, -6.1, 6.1}; | ||
| AxisSpec ZAxis = {301, -30.1, 30.1}; | ||
| AxisSpec PhiAxis = {629, 0, 2 * M_PI, "Rad", "phi axis"}; | ||
| AxisSpec PhiAxis = {629, 0, 2 * o2::constants::math::PI, "Rad", "phi axis"}; |
There was a problem hiding this comment.
Please replace 2. * o2::constants::math::PI by o2::constants::math::TwoPI
Please consider the following formatting changes to AliceO2Group#11805
abmodak
left a comment
abmodak
left a comment
There was a problem hiding this comment.
Please consider the following changes
for (auto& bc : bcs) => for (const auto& bc : bcs)
for (auto& collision : collisions) => for (const auto& collision : collisions)
for (auto& track : tracks) => for (const auto& track : tracks)
for (auto& retrack : retracks) => for (const auto& retrack : retracks)
for (auto& particle : perCollisionMCSample) => for (const auto& particle : perCollisionMCSample)
for (auto& particle : perCollisionMCSampleCentral) => for (const auto& particle : perCollisionMCSampleCentral)
Please consider the following formatting changes to AliceO2Group#11805
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
3 participants
No description provided.