Skip to content
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.

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

PWGHF : syncing decays in the preselection with enums used for MC #5223

Merged
merged 1 commit into from Jan 19, 2021

Conversation

nzardosh
Copy link
Contributor

No description provided.

@@ -226,52 +228,78 @@ struct HFTrackIndexSkimsCreator {
}

//FIXME move above process function
const int n2ProngDecays = 2; //number of two prong hadron types
const int n3ProngDecays = 3; //number of three prong hadron types
const int n2ProngDecays = N2ProngDecays; //number of two prong hadron types
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you need to redefine it? Can't you use directly N2ProngDecays?

@jgrosseo jgrosseo merged commit 5cec71a into AliceO2Group:dev Jan 19, 2021
{"hmassLc", "Lc candidates;inv. mass (p K #pi) (GeV/#it{c}^{2});entries", {HistType::kTH1F, {{500, 0., 5.}}}},
{"hmassDs", "Ds candidates;inv. mass (K K #pi) (GeV/#it{c}^{2});entries", {HistType::kTH1F, {{500, 0., 5.}}}}}};
{"hmassDPlusToPiKPi", "D+ candidates;inv. mass (#pi K #pi) (GeV/#it{c}^{2});entries", {HistType::kTH1F, {{500, 0., 5.}}}},
{"hmassLctoPKPi", "Lc candidates;inv. mass (p K #pi) (GeV/#it{c}^{2});entries", {HistType::kTH1F, {{500, 0., 5.}}}},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a typo. It should be hmassLcToPKPi.
@jgrosseo How come that none of the builds complained?

EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants