Skip to content

[PWGDQ] Please consider the following formatting changes to #8809 #15

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

[PWGDQ] Please consider the following formatting changes to #8809 #15

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 PWGDQ/TableProducer/tableMakerMC.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
// The skimmed MC stack includes the MC truth particles corresponding to the list of user specified MC signals (see MCsignal.h)
// and the MC truth particles corresponding to the reconstructed tracks selected by the specified track cuts on reconstructed data.

#include <iostream>

Check warning on line 19 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[include-iostream] 

Including iostream is discouraged. Use O2 logging instead.
#include <string>
#include <map>
#include <memory>
Expand Down Expand Up @@ -933,8 +933,8 @@
mothers.push_back(fNewLabels.find(m)->second);
}
} else {
cout << "Mother label (" << m << ") exceeds the McParticles size (" << mcTracks.size() << ")" << endl;

Check warning on line 936 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
cout << " Check the MC generator" << endl;

Check warning on line 937 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
}
}
}
Expand All @@ -950,8 +950,8 @@
daughters.push_back(fNewLabels.find(d)->second);
}
} else {
cout << "Daughter label (" << d << ") exceeds the McParticles size (" << mcTracks.size() << ")" << endl;

Check warning on line 953 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
cout << " Check the MC generator" << endl;

Check warning on line 954 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
}
}
}
Expand Down Expand Up @@ -1231,7 +1231,7 @@
// check all the specified signals and fill histograms for MC truth matched tracks
for (auto& sig : fMCSignals) {
if (sig.CheckSignal(true, mctrack)) {
mcflags |= (static_cast<uint16_t>(1)> << i);
mcflags |= (static_cast<uint16_t>(1) > << i);
if (fDoDetailedQA) {
j = 0;
for (auto& cut : fTrackCuts) {
Expand Down Expand Up @@ -1475,8 +1475,8 @@
mothers.push_back(fNewLabels.find(m)->second);
}
} else {
cout << "Mother label (" << m << ") exceeds the McParticles size (" << mcTracks.size() << ")" << endl;

Check warning on line 1478 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
cout << " Check the MC generator" << endl;

Check warning on line 1479 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
}
}
}
Expand All @@ -1492,8 +1492,8 @@
daughters.push_back(fNewLabels.find(d)->second);
}
} else {
cout << "Daughter label (" << d << ") exceeds the McParticles size (" << mcTracks.size() << ")" << endl;

Check warning on line 1495 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
cout << " Check the MC generator" << endl;

Check warning on line 1496 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[logging] 

Consider using O2 logging (LOG, LOGF, LOGP).
}
}
}
Expand Down Expand Up @@ -1529,7 +1529,7 @@
void DefineHistograms(TString histClasses)
{
std::unique_ptr<TObjArray> objArray(histClasses.Tokenize(";"));
for (Int_t iclass = 0; iclass < objArray->GetEntries(); ++iclass) {

Check warning on line 1532 in PWGDQ/TableProducer/tableMakerMC.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[root-entity] 

Consider replacing ROOT entities with equivalents from standard C++ or from O2.
TString classStr = objArray->At(iclass)->GetName();
if (fConfigQA) {
fHistMan->AddHistClass(classStr.Data());
Expand Down
Loading