-
Notifications
You must be signed in to change notification settings - Fork 327
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
WeightCalculatorFromHistogram.cc gives NaN #119
Comments
I opened a PR about this almost 10 months ago: #87 |
@bianchini can you confirm that #87 works for you and I will merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/cms-nanoAOD/nanoAOD-tools/blob/master/src/WeightCalculatorFromHistogram.cc#L101
initiializes 'cropped' once and then it fills it with while() as many times as needed by thw whil() condition, but checkIntegral() assumes that the length stays the same, so it may give NaN when accessing elements beyond refvals_.size().
The text was updated successfully, but these errors were encountered: