-
Notifications
You must be signed in to change notification settings - Fork 466
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
The gen <=> reco mapping tests task #68
Conversation
For the Run 3 productions still needs the TrackExtensionTask being patched
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comments (one of them ">=" needs to be fixed). The rest is up to you.
Also few cosmetic changes and removing the conversion to integer charge before counting generated particles
@@ -1502,7 +1502,7 @@ struct CheckGeneratorLevelVsDetectorLevel { | |||
for (auto& part : mcParticles) { | |||
auto pdgpart = fPDG->GetParticle(part.pdgCode()); | |||
if (pdgpart != nullptr) { | |||
float charge = (pdgpart->Charge() / 3 >= 1) ? 1.0 : ((pdgpart->Charge() / 3 <= -1) ? -1.0 : 0.0); | |||
float charge = (pdgpart->Charge() >= 3) ? 1.0 : ((pdgpart->Charge() <= -3) ? -1.0 : 0.0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merge it now, but these three lines I would have just written
if (pdgpart->Charge() != 0) {
ngen++
}
there will be no quarks I assume :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will not be the first time I've found the quarks at generator level
Thanks anyway for the suggestion
And for the fixes!
Please consider the following formatting changes to AliceO2Group#6380
For the Run 3 productions still needs the TrackExtensionTask being patched