-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implementation of ATLAS_WJ_8TEV in the new format #2210
Conversation
183f7a5
to
809263e
Compare
I have implemented the two variants I discussed in the description - default (symmetrized) and |
nnpdf_data/nnpdf_data/commondata/ATLAS_WJ_8TEV/uncertainties_WP-PT.yaml
Outdated
Show resolved
Hide resolved
What do you mean they should be equal? If they are not, does that means there's a bug in the old, in the new...? |
I mean that the legacy implementation adopted the cms prescription to treat the asymmetric uncertainties as default. Hence, the variant
|
Ok, then they should not be equal, right? |
Yes, they should not be equal |
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.
So, the implementation itself I don't see anything wrong however, I'm slightly worried about the chi2 changes. The CMS prescription should've kept the results at least similar, however the change in the chi2 is huge (from ~1 to ~1.5), in the "default" it goes all the way to chi2 ~ 3... and you left a few TODO
in the filter_utils and filter files. Are those to-do no longer applicable or some of those checks would change the results?
@enocera is this expected? I am happy with merging this as it is and revising possible problems in the future if you are ok with it.
@achiefa please, remove the old kinematics file
You're right. I'll check whether these TODO's affect the results. Thank you for pointing this out. |
@achiefa @scarlehoff This is unexpected. @achiefa If I understand correctly, there are two versions on Hepdata. what happens if you use the other version? |
@enocera It doesn't improve with the older version either. BTW, I was looking carefully at the single sources of systematic uncertainties and I observed that, in some cases, the plus-minus signs are inverted (see the snapshot below). Do these uncertainties require a particular treatment? |
@achiefa The case that you have highlighted has symmetric uncertainties upside down. In this case you should also use the symmetrisation formula by D'Agostini, wich will return a "symmetrised" uncertainty of -4 (and not +4). This means that the uncertainty is anticorrelated. Are you sure that the sign is correctly taken into account? |
Ok, I think I was mistakenly taking the absolute value of some numbers. Now the symmetrized version (which is going to become the default variant in the new implementation) gives much better results [see here]. |
Great! This is much better. Not only that, it is close to the numbers one gets with the |
Indeed! |
Wait, the link to |
Ok, updated. Honestly, I don't see much difference between |
Ok, now I have uploaded the data-theory comparisons for version 1 and version 2 of the HepData tables. |
e4f030b
to
f9513ff
Compare
Nice. Is this ready then? |
Yes, it is. The inter-experiment correlations are still missing, but my understanding is that we will address them in a second iteration. Maybe it is worth making a note pointing to this PR for reference. In the description I put all the necessary information to include these correlations. |
f9513ff
to
c3756f1
Compare
Perfect! Thanks |
Sorry, just one question, then the |
The CMS prescription was the one adopted in the legacy implementation and I didn't manage to reproduce it. However, after a chat with @enocera, we agreed to neglect this prescription so that its implementation is entirely unnecessary. |
Re-implementation of the ATLAS_WJ_8TEV in the new commondata format. I summarise the relevant information below, also as a reminder for myself.
Legacy reports: [legacy], [legacy_ATLAS], [legacy_NP]
New reports: [default HepData v1], [default HepData v2]
default = symmetrized,
Description
This dataset provides a correlation matrix for statistical uncertainties. Systematic uncertainties are given for diagonal entries only. Quoting from the ATLAS paper:
The statistical uncertainties provided in the HepData table are:
Please, @enocera @scarlehoff, feel free to check if this breakdown has flaws, as matching with the paper was not trivial.
There are a few things to mention:
ATLASWX
).For the record, there is a third variant in the legacy implementation, which accounts for non-perturbative corrections (NP) and that I will ignore for the moment since they were not used in NNPDF4.0.