Skip to content

[PWGJE] Fix includes #16414

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

Open
vkucera wants to merge 1 commit into
base: master
Choose a base branch
from
+3 −2
Open

[PWGJE] Fix includes #16414

Show file tree
Hide file tree
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: 0 additions & 2 deletions PWGJE/Tasks/bjetTaggingGnn.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@
#include "Common/Core/Zorro.h"
#include "Common/Core/ZorroSummary.h"
#include "Common/DataModel/EventSelection.h"
#include "Common/DataModel/Multiplicity.h"
#include "Common/DataModel/TrackSelectionTables.h"

#include <CCDB/BasicCCDBManager.h>
#include <Framework/ASoA.h>
Expand Down
1 change: 1 addition & 0 deletions PWGJE/Tasks/gammaJetTreeProducer.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@

#include <algorithm>
#include <bitset>
#include <cstddef>
#include <cstdint>
#include <string>
#include <type_traits>
Expand Down Expand Up @@ -288,13 +289,13 @@
trackSourceIndex.push_back(originalIndex);
if (phi <= (maxMatchingDistance + additionalMargin)) {
trackEta.push_back(track.eta());
trackPhi.push_back(phi + o2::constants::math::TwoPI);

Check failure on line 292 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[two-pi-add-subtract] 

Use RecoDecay::constrainAngle to restrict angle to a given range.
trackPt.push_back(track.pt());
trackSourceIndex.push_back(originalIndex);
}
if (phi >= (o2::constants::math::TwoPI - (maxMatchingDistance + additionalMargin))) {
trackEta.push_back(track.eta());
trackPhi.push_back(phi - o2::constants::math::TwoPI);

Check failure on line 298 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[two-pi-add-subtract] 

Use RecoDecay::constrainAngle to restrict angle to a given range.
trackPt.push_back(track.pt());
trackSourceIndex.push_back(originalIndex);
}
Expand Down Expand Up @@ -337,13 +338,13 @@
mcParticleSourceIndex.push_back(originalIndex);
if (phi <= (maxMatchingDistance + additionalMargin)) {
mcParticleEta.push_back(particle.eta());
mcParticlePhi.push_back(phi + o2::constants::math::TwoPI);

Check failure on line 341 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[two-pi-add-subtract] 

Use RecoDecay::constrainAngle to restrict angle to a given range.
mcParticlePt.push_back(particle.pt());
mcParticleSourceIndex.push_back(originalIndex);
}
if (phi >= (o2::constants::math::TwoPI - (maxMatchingDistance + additionalMargin))) {
mcParticleEta.push_back(particle.eta());
mcParticlePhi.push_back(phi - o2::constants::math::TwoPI);

Check failure on line 347 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[two-pi-add-subtract] 

Use RecoDecay::constrainAngle to restrict angle to a given range.
mcParticlePt.push_back(particle.pt());
mcParticleSourceIndex.push_back(originalIndex);
}
Expand Down Expand Up @@ -583,7 +584,7 @@
int selectedMother = -1;
if (mothers.size() == 1) {
selectedMother = 0;
} else if (mothers.size() == 2 &&

Check failure on line 587 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
mothers[0].globalIndex() == mothers[1].globalIndex() &&
mothers[0].pdgCode() == mothers[1].pdgCode() &&
mothers[0].getGenStatusCode() == mothers[1].getGenStatusCode()) {
Expand Down Expand Up @@ -614,7 +615,7 @@
int selectedMother = -1;
if (mothers.size() == 1) {
selectedMother = 0;
} else if (mothers.size() == 2) {

Check failure on line 618 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
twoMothersIdentical = (mothers[0].globalIndex() == mothers[1].globalIndex() &&
mothers[0].pdgCode() == mothers[1].pdgCode() &&
mothers[0].getGenStatusCode() == mothers[1].getGenStatusCode());
Expand All @@ -630,7 +631,7 @@
twoMothersIdentical = false;
if (mothers.size() == 1) {
selectedMother = 0;
} else if (mothers.size() == 2) {

Check failure on line 634 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
twoMothersIdentical = (mothers[0].globalIndex() == mothers[1].globalIndex() &&
mothers[0].pdgCode() == mothers[1].pdgCode() &&
mothers[0].getGenStatusCode() == mothers[1].getGenStatusCode());
Expand All @@ -647,7 +648,7 @@
/// \return true if particle is a prompt photon, false otherwise
bool isPromptPhoton(const auto& particle)
{
if (particle.pdgCode() == PDG_t::kGamma && particle.isPhysicalPrimary() && std::abs(particle.getGenStatusCode()) < 90) {

Check failure on line 651 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[magic-number] 

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
return true;
}
return false;
Expand Down Expand Up @@ -718,7 +719,7 @@
// check if it has mothers that are etas
const auto& mothers = particle.template mothers_as<aod::JMcParticles>();
for (const auto& mother : mothers) {
if (mother.pdgCode() == 221) {

Check failure on line 722 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[pdg/explicit-code] 

Avoid hard-coded PDG codes. Use named values from PDG_t or o2::constants::physics::Pdg instead.
return true;
}
}
Expand All @@ -735,7 +736,7 @@
// check if you find a pi0 mother or a eta mother
const auto& mothers = particle.template mothers_as<aod::JMcParticles>();
for (const auto& mother : mothers) {
if (mother.pdgCode() == PDG_t::kPi0 || mother.pdgCode() == 221) {

Check failure on line 739 in PWGJE/Tasks/gammaJetTreeProducer.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[pdg/explicit-code] 

Avoid hard-coded PDG codes. Use named values from PDG_t or o2::constants::physics::Pdg instead.
return false;
}
}
Expand Down
1 change: 1 addition & 0 deletions PWGJE/Tasks/jetSpectraEseTask.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@

#include <TF1.h>
#include <TH1.h>
#include <TH3.h>
#include <THnSparse.h>
#include <TMath.h>
#include <TRandom3.h>
Expand Down
1 change: 1 addition & 0 deletions PWGJE/Tasks/jetSubstructureHF.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

#include "PWGJE/Core/FastJetUtilities.h"
#include "PWGJE/Core/JetDQUtilities.h"
#include "PWGJE/Core/JetDerivedDataUtilities.h"
#include "PWGJE/Core/JetFinder.h"
#include "PWGJE/Core/JetFindingUtilities.h"
#include "PWGJE/Core/JetHFUtilities.h"
Expand Down
Loading