-
Notifications
You must be signed in to change notification settings - Fork 7
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
Implementing ATLAS_PH-8TEV_XSEC #2246
base: master
Are you sure you want to change the base?
Conversation
Hi @RoyStegeman , I forgot how we agreed to proceed here at the last code meeting. Do you remember it? Sorry! |
It was suggested to transform the APPLgrid to pineappl format and add it to the new theory (708) |
Hi @enocera, I noticed that the old buildmaster implementation of this dataset doesn't provide the applgrid for the rapidity region 1.81<|eta(gamma)| < 2.37. It isn't listed at https://github.com/NNPDF/applgrids/tree/master/ATLASPHT12, while the 13 TeV counterpart does provide the applgrid in this region. Do you recall the reason for this? The hepdata entry is https://www.hepdata.net/record/ins1457605?version=1&table=Table%204 for reference. |
Dear @jacoterh as far as I can recollect, the reason is historical. Prompt photon data (8 TeV and 13 teV) were originally studied in https://arxiv.org/pdf/1802.03021. The NNPDF4.0 implementation closely follows the results in that paper. On page 4, one can read: "the most forward rapidity bin, 1.81 ≤ |ηγ | < 2.37, is excluded from the fit due to the tensions between the experimental data and the theoretical predictions." I guess that, for this reason, the PineAPPL grids were not generated at all in NNPDF4.0. At 13 TeV, the paper does not report any restriction. |
For now, I suggest to assume that this statement (and the NNPDF4.0 selection) hold true. |
@jacoterh note that it is possible to shift a bin for a number of points like in this case:
In this example |
Thanks @enocera, that clarifies it! |
This PR implements ATLAS prompt photon production at 8 TeV, similar to #2209 at 13 TeV. No legacy implementation exists for this dataset as it wasn't ported from buildmaster.
I tried to produce a validphys report, but it complains it cannot use an old theory with a purely new dataset:
[ERROR]: Bad configuration encountered: Cannot use an old theory with a purely new dataset (ATLAS_PH_8TEV_XSEC)
I used this card.