Skip to content

Add theory cards for hadronic N3LO QCD, theory ID 41000100#2467

Open
kamillaurent wants to merge 5 commits into
masterfrom
add_theory_cards_N3LO
Open

Add theory cards for hadronic N3LO QCD, theory ID 41000100#2467
kamillaurent wants to merge 5 commits into
masterfrom
add_theory_cards_N3LO

Conversation

@kamillaurent
Copy link
Copy Markdown

This PR is intended to add the theory_cards for generating N3LO QCD FK Tables of hadronic processes. The theory ID of the central scale is 41000100, so the added files are 410001*.yaml.

@kamillaurent kamillaurent requested a review from felixhekhorn May 11, 2026 13:26
@felixhekhorn felixhekhorn requested a review from scarlehoff May 11, 2026 13:45
Copy link
Copy Markdown
Contributor

@felixhekhorn felixhekhorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • I think the agreement is - and that seems to be the case on master - to NOT commit the "sub-theory" cards, i.e. the cards for the individual FONLL runs, so you should remove those; e.g. 41_000_000_00
  • I see this is also the case for 41_000_000 , but why is there a gap in numbers to the SV cards? i.e. central = 00, SV = 10, 11, 12, etc. so why not 01, 02, etc.? there might be a legitimate reason

@scarlehoff
Copy link
Copy Markdown
Member

Indeed. Please do not commit the nfonll runcards.

@kamillaurent
Copy link
Copy Markdown
Author

I removed the nfonll runcards, @felixhekhorn I assume the gap in numbers is to avoid confusion with the FONLL runcards, which are {01..06}, I used {10..15} following the convention used for the scale variation of 41000000.

@felixhekhorn
Copy link
Copy Markdown
Contributor

I removed the nfonll runcards

dito for 41_041_000_00 and friends

Comment thread nnpdf_data/nnpdf_data/theory_cards/.gitignore Outdated
@scarlehoff scarlehoff force-pushed the add_theory_cards_N3LO branch from 7e561e6 to 837ffda Compare May 12, 2026 09:27
@scarlehoff
Copy link
Copy Markdown
Member

Thanks. I've rebased and will merge once tests pass.

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