Conversation
|
I have a few questions regarding this dataset: Regarding this dataset, should I change figure_by from 'eta' to 'radius' in the metadata file? Given that the two subfigures I'm implementing are differentiated by radius. Also, should I add 'radius' as a variable in the kinematic coverage? Finally, is there not a rapidity bin missing? namely 1.37 < eta < 1.56? For reference, the HEPData link: https://www.hepdata.net/record/ins2628741 And the figure:
|
For the kinematic variables, Then, regarding the rest. I'd say these are either two different observables or two variants (@enocera should decide), but the radius is not part of the kinematic variables. @enocera another question, does this measurement makes the other 13 TeV photon that we have obsolete? i.e., when running grids, do you want me to focus on this one? |
|
@vschutze-alt (cc @scarlehoff ) Concering the radius, I think that here the situation is similar to that we have with single-inclusive jet or di-jet production, in which indeed we can have two versions of the same data set that differ for the jet radius. In that case we have implemented two variants of the same data set, therefore I would do the same here. To have an idea of how a variant works in this case, you can have a look at |
0212b69 to
54345c4
Compare
enocera
left a comment
There was a problem hiding this comment.
Dear @VLTSML26 I have revised this PR. I've made the following alterations.
metadata.yaml: I have corrected a typo in the journal name; I have changed the names of the observables; I have added the c.m. energy (even if not required) in the kinematic specifications; I have added the (provisional) theory block.data*,kinematics*anduncertainteis*files: the names are modified according to the new names of the observables; note that, after a careful look at the paper, I came to a slight different conclusion on the uncorrelated systematic uncertainties. Note that I set all the uncorrelated uncertainties to theADDtreatment. They indeed look of statistical origin, therefore it seems more appropriate for them to be additive. I have left a comment about that in filter.py.filter.py: I have changed a little the way in which the data is parsed, and I have implemented the new names for the observables.
Note that tests are now passing (I suspect that there was a kinematic mismatch).
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
@enocera @felixhekhorn Alex has sent me the NNLOJET runcard they used in the paper so in principle it should be possible to regenerate, however, they have photon fragmentation which introduces a new scale
The possibilities I think are:
- Using Frixione's algorithm to avoid the need for the fragmentation (which Sherpa does in the same paper), will require a bit of tweaking but having already a runcard it should be doable.
- Choosing the same central scale as in the paper and hoping for the best
- 2 + add an extra scale-uncertainty
For the time being I'll go with 2
scarlehoff
left a comment
There was a problem hiding this comment.
I only have the NLO for now, the NNLO is still running, it might be ready by Amsterdam but in any case:
Somehow the chi2 is great even if there's a 10% difference in the data-th comparison still:
Exp chi2: 1.274
This is for the R=0.4. I'll run the R=0.2 afterwards. But I think given this NLO this can be merged (upon fixing a bit the labels, the title didn't work because of a combination of unescaped \ and latex).
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_PH_13TEV_139fb/metadata.yaml
Outdated
Show resolved
Hide resolved
…d uncertainty variants; added c.m. energy
…ncertainties; added bin limits; added c.m. energy, even if not needed
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
…d uncertainty variants; added c.m. energy
…ncertainties; added bin limits; added c.m. energy, even if not needed
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
….yaml Co-authored-by: Juan M. Cruz-Martinez <juacrumar@lairen.eu>
3bb5b68 to
2dbf5aa
Compare
|
I'm doing a few more tests for the R=0.4 cone but I'm not entirely sure what the issue with the NNLO is. The problem seem to be isolated to the first few bins in rapidity (the ones before the cap) so I might just rerun one of those bins in insolation to understand it. But I'd suggest taking the R=0.2 version as the good one for now (and for the data selection) |

No description provided.