Skip to content
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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Implementing ATLAS_PH-8TEV_XSEC #2246

wants to merge 3 commits into from

Conversation

jacoterh
Copy link
Collaborator

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.

@jacoterh
Copy link
Collaborator Author

Hi @RoyStegeman , I forgot how we agreed to proceed here at the last code meeting. Do you remember it? Sorry!

@RoyStegeman
Copy link
Member

RoyStegeman commented Dec 13, 2024

It was suggested to transform the APPLgrid to pineappl format and add it to the new theory (708)

@jacoterh
Copy link
Collaborator Author

jacoterh commented Feb 4, 2025

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.

@enocera
Copy link
Contributor

enocera commented Feb 4, 2025

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.

@enocera
Copy link
Contributor

enocera commented Feb 4, 2025

For now, I suggest to assume that this statement (and the NNPDF4.0 selection) hold true.

@scarlehoff
Copy link
Member

@jacoterh note that it is possible to shift a bin for a number of points like in this case:

In this example ATLASZPT8TEVMDIST-ATLASZPT8TEV-MLLBIN4_ptZ and ATLASZPT8TEVMDIST-ATLASZPT8TEV-MLLBIN6_ptZ both get shifted by 10 to account for the gap between the data and the theory.

@jacoterh
Copy link
Collaborator Author

jacoterh commented Feb 4, 2025

Thanks @enocera, that clarifies it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants