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

Convert daughters branches into double precision #53

Open
l-cadamuro opened this issue Sep 17, 2016 · 1 comment
Open

Convert daughters branches into double precision #53

l-cadamuro opened this issue Sep 17, 2016 · 1 comment

Comments

@l-cadamuro
Copy link
Contributor

Float precision used for daughters_ branches (and maybe mothers_ as well) causes the mass values computed downstream using TLorentzVector to be different from the original value.
Probably not an issue because nobody uses the mass as a variable, but appears as differences in the synchronization. Note that this does not affect SVFit computation.
To be changed to double after the rest of the downstream codes are ready to handle vectors of doubles.

@francescobrivio
Copy link

Spent some time today trying to understand why the mass of the daughters was not as the original value in miniAODs, I see you already knew since 4 years @l-cadamuro :) I'll leave this open in the hope that we will find time to update the downstream code in the near future!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants