Skip to content

Commit 1cb72bf

Browse files
committed
Fix C++ data types according to PR review
1 parent dd70059 commit 1cb72bf

File tree

4 files changed

+7
-8
lines changed

4 files changed

+7
-8
lines changed

PWGHF/DataModel/CandidateReconstructionTables.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1768,7 +1768,7 @@ namespace hf_cand_xic_to_xi_pi_pi
17681768
{
17691769
DECLARE_SOA_INDEX_COLUMN_FULL(Pi0, pi0, int, Tracks, "_pi0");
17701770
DECLARE_SOA_INDEX_COLUMN_FULL(Pi1, pi1, int, Tracks, "_pi1");
1771-
DECLARE_SOA_COLUMN(Sign, sign, float);
1771+
DECLARE_SOA_COLUMN(Sign, sign, int8_t);
17721772
DECLARE_SOA_COLUMN(InvMassXicPlus, invMassXicPlus, float);
17731773
DECLARE_SOA_COLUMN(InvMassXi, invMassXi, float);
17741774
DECLARE_SOA_COLUMN(InvMassLambda, invMassLambda, float);

PWGHF/TableProducer/candidateCreatorXicToXiPiPi.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ struct HfCandidateCreatorXicToXiPiPi {
300300

301301
//----------------------------calculate physical properties-----------------------
302302
// Charge of charm baryon
303-
int signXic = casc.sign() < 0 ? +1 : -1;
303+
int8_t signXic = casc.sign() < 0 ? +1 : -1;
304304

305305
// get SV properties
306306
const auto& secondaryVertex = df.getPCACandidate();
@@ -561,7 +561,7 @@ struct HfCandidateCreatorXicToXiPiPi {
561561

562562
//---------------------calculate physical parameters of XicPlus candidate----------------------
563563
// sign of charm baryon
564-
int signXic = casc.sign() < 0 ? +1 : -1;
564+
int8_t signXic = casc.sign() < 0 ? +1 : -1;
565565

566566
// transport XicPlus daughters to XicPlus decay vertex (secondary vertex)
567567
float secondaryVertex[3] = {0.};

PWGHF/TableProducer/candidateSelectorXicToXiPiPi.cxx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,6 @@ struct HfCandidateSelectorXicToXiPiPi {
141141
}
142142

143143
if (fillHistogram) {
144-
constexpr int kNBinsSelectionCriteria = NSelectionCriteria;
145144
std::string labels[NSelectionCriteria];
146145
labels[All] = "All";
147146
labels[Pt] = "#it{p}_{T}";
@@ -160,13 +159,13 @@ struct HfCandidateSelectorXicToXiPiPi {
160159
labels[BdtSelected] = "BDT selection";
161160

162161
if (doprocessData) {
163-
registry.add("hSelCandidates", ";;entries", {HistType::kTH1F, {{kNBinsSelectionCriteria, -0.5f, static_cast<float>(kNBinsSelectionCriteria) - 0.5f}}});
162+
registry.add("hSelCandidates", ";;entries", {HistType::kTH1F, {{NSelectionCriteria, -0.5f, static_cast<float>(NSelectionCriteria) - 0.5f}}});
164163
for (int iBin = 0; iBin < NSelectionCriteria; ++iBin) {
165164
registry.get<TH1>(HIST("hSelCandidates"))->GetXaxis()->SetBinLabel(iBin + 1, labels[iBin].data());
166165
}
167166
} else if (doprocessMc) {
168-
registry.add("hSelCandidatesRecSig", ";;entries", {HistType::kTH1F, {{kNBinsSelectionCriteria, -0.5f, static_cast<float>(kNBinsSelectionCriteria) - 0.5f}}});
169-
registry.add("hSelCandidatesRecBkg", ";;entries", {HistType::kTH1F, {{kNBinsSelectionCriteria, -0.5f, static_cast<float>(kNBinsSelectionCriteria) - 0.5f}}});
167+
registry.add("hSelCandidatesRecSig", ";;entries", {HistType::kTH1F, {{NSelectionCriteria, -0.5f, static_cast<float>(NSelectionCriteria) - 0.5f}}});
168+
registry.add("hSelCandidatesRecBkg", ";;entries", {HistType::kTH1F, {{NSelectionCriteria, -0.5f, static_cast<float>(NSelectionCriteria) - 0.5f}}});
170169
for (int iBin = 0; iBin < NSelectionCriteria; ++iBin) {
171170
registry.get<TH1>(HIST("hSelCandidatesRecSig"))->GetXaxis()->SetBinLabel(iBin + 1, labels[iBin].data());
172171
registry.get<TH1>(HIST("hSelCandidatesRecBkg"))->GetXaxis()->SetBinLabel(iBin + 1, labels[iBin].data());

PWGHF/TableProducer/treeCreatorXicToXiPiPi.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ namespace o2::aod
4343
{
4444
namespace full
4545
{
46-
DECLARE_SOA_COLUMN(ParticleFlag, particleFlag, int); //! hf_cand_xic_to_xi_pi_pi::Sign for data, hf_cand_xic_to_xi_pi_pi::FlagMcMatchRec for MC
46+
DECLARE_SOA_COLUMN(ParticleFlag, particleFlag, int8_t); //! hf_cand_xic_to_xi_pi_pi::Sign for data, hf_cand_xic_to_xi_pi_pi::FlagMcMatchRec for MC
4747
DECLARE_SOA_COLUMN(CandidateSelFlag, candidateSelFlag, int); //! Selection flag of candidate (output of candidateSelector)
4848
// vertices
4949
DECLARE_SOA_COLUMN(Chi2SV, chi2SV, float); //! Chi2 of candidate vertex

0 commit comments

Comments
 (0)