Skip to content

ATLAS and CMS W/Charm#704

Merged
siranipour merged 12 commits into
masterfrom
CMS_WCHARM_13TEV
Apr 13, 2020
Merged

ATLAS and CMS W/Charm#704
siranipour merged 12 commits into
masterfrom
CMS_WCHARM_13TEV

Conversation

@enocera
Copy link
Copy Markdown
Contributor

@enocera enocera commented Apr 5, 2020

This PR contains the data for the following experiments:

  • CMS W+charm rapidity differential distributions, 13 TeV (NLO)
  • ATLAS W+charm rapidity differential distributions, 7 TeV (NLO).

The necessary buildmaster (nnpdf/buildmaster#135 and nnpdf/buildmaster#82), external (NNPDF/external#56), applgrids (NNPDF/applgrids#30) and apfelcomb (NNPDF/apfelcomb#68) PRs have already been merged in the corresponding repos. #675 has been updated accordingly.

@enocera enocera requested a review from siranipour April 5, 2020 10:12
@siranipour
Copy link
Copy Markdown
Contributor

Hi @enocera, this looks good to me. Shall we add the commondata for CMS Z+C at 8TeV in this PR too? nnpdf/buildmaster#83

@siranipour
Copy link
Copy Markdown
Contributor

Ah, we're probably waiting on the APPLgrids for this right?

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented Apr 6, 2020

Indeed, we don't have the applgrids for Z+charm yet. I'd prefer to have a separate PR for that data set once it will be ready.

@siranipour
Copy link
Copy Markdown
Contributor

Ok that makes sense! Looks all good to me Emanuele! This week is devoted to the theory side of these datasets btw

@siranipour
Copy link
Copy Markdown
Contributor

Btw, where do we update the theory 53 or indeed the new FKTables when we add new datasets

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented Apr 6, 2020

Btw, where do we update the theory 53 or indeed the new FKTables when we add new datasets

Good question. The theories are stored at nnpdf@lxplus.cern.ch. There's a script in apfelcomb that uploads a new theory automatically. This is documented here. The script MUST be used with CARE, especially if you want to upload the FK tables of a single data set for an existing theory.

@siranipour
Copy link
Copy Markdown
Contributor

Ah I see, many thanks for the link Emanuele

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented Apr 6, 2020

Ok that makes sense! Looks all good to me Emanuele! This week is devoted to the theory side of these datasets btw.

The thing I'd start from is to run madgraph with iappl=0 (no generation of the applgrids) and check that you can reproduce the results in Fig.11 of https://arxiv.org/abs/1711.02143. If so, then we can wonder about producing applgrids and generating FK tables.

@scarrazza
Copy link
Copy Markdown
Member

If we can merge this, could you please rebase to master?

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented Apr 6, 2020

@scarrazza Rebased. Checks are now failing because "The job was not started because the spending limit for Actions and Packages has been exceeded."

@scarrazza
Copy link
Copy Markdown
Member

Perfect thanks for checking, tomorrow this should be working.

@siranipour siranipour changed the title Cms wcharm 13 tev ATLAS and CMS W/Charm Apr 13, 2020
@siranipour
Copy link
Copy Markdown
Contributor

Tests passed, merging this now.

@siranipour siranipour merged commit 1b3a414 into master Apr 13, 2020
@siranipour siranipour deleted the CMS_WCHARM_13TEV branch April 13, 2020 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants