-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added CDFZRAP_NEW data set #79
Conversation
I think the
which doesn't match the bin numbers in the database and don't match the number of K factors. |
@cschwan Thanks for checking. This is strange. Let me try to proceed one step at a time.
Unless I'm missing something, I don't think that there's a mismatch (and if there were one, I wouldn't be able to successfully produce the validphys data/theory comparison in the first place). Maybe I didn't understand what you meant? |
@enocera :
What I'm worried about is there is a mismatch, which can only show up in the last bin, since the theory prediction are the same otherwise. |
@cschwan OMG! You're right! THERE IS a mismatch in the abpfelcomb database, the .root file in 500 should be in 501 and the other way around! Thanks so much! Let me fix this! |
Everything works by accident because apfelcomb checks only the number of bins, but not the bin kinematics! |
Well, 501 works, but not 500. |
Great! I was very surprised by the fit of the last bin; before the fix the data is too large, and now it was too small. I hope that with the right theory prediction fit and data are much closer. |
This PR extends the database to the generation of a fixed version of the CDF Z rapidity distributions.