Skip to content

4.1 N3LO theory cards#2494

Open
jekoorn wants to merge 12 commits into
masterfrom
theory_cards_41
Open

4.1 N3LO theory cards#2494
jekoorn wants to merge 12 commits into
masterfrom
theory_cards_41

Conversation

@jekoorn

@jekoorn jekoorn commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Write the N3LO theory cards for:

  • central theory
  • anomalous dimensions variations
  • scale variations
  • DIS coeff functions fact variations
  • IHOU + MHOU missing prescription
  • alpha_s variations

And:

  • write the scalevariationtheoryids.yaml accordingly

Comment thread nnpdf_data/nnpdf_data/theory_cards/41300040.yaml Outdated
Comment thread nnpdf_data/nnpdf_data/theory_cards/41300038.yaml Outdated
Comment thread nnpdf_data/nnpdf_data/theory_cards/41300000.yaml Outdated
Comment thread nnpdf_data/nnpdf_data/theory_cards/41300000.yaml Outdated
Comment thread nnpdf_data/nnpdf_data/theory_cards/41300010.yaml Outdated

@Radonirinaunimi Radonirinaunimi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pretty much overlaps with what @felixhekhorn already mentioned. The main point being the use of FHMRUVV parametrization as baseline.

Comment thread nnpdf_data/nnpdf_data/theory_cards/41300000.yaml Outdated
Comment thread validphys2/src/validphys/scalevariations/scalevariationtheoryids.yaml Outdated
@felixhekhorn

Copy link
Copy Markdown
Contributor

Let's remember to make a pinefarm release with the new yadism version, @jekoorn please take care of that.

cc @scarlehoff

@felixhekhorn

Copy link
Copy Markdown
Contributor

Please resolve comments, which you believe to have addressed so we know what to worry about

@felixhekhorn

Copy link
Copy Markdown
Contributor

Let's remember to make a pinefarm release with the new yadism version, @jekoorn please take care of that.

NNPDF/pinefarm#131

@jekoorn

jekoorn commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Let's remember to make a pinefarm release with the new yadism version, @jekoorn please take care of that.

NNPDF/pinefarm#131

New release
Should we create a new release from the tag? I see there are no existing proper releases yet

@Radonirinaunimi Radonirinaunimi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some minor comments/requests.

Comment thread validphys2/src/validphys/scalevariations/scalevariationtheoryids.yaml Outdated

@Radonirinaunimi Radonirinaunimi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry, I missed this before.

Comment thread nnpdf_data/nnpdf_data/theory_cards/41042014.yaml Outdated

@Radonirinaunimi Radonirinaunimi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good to me now.

Comment thread nnpdf_data/nnpdf_data/theory_cards/41042011.yaml Outdated

@felixhekhorn felixhekhorn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image there is something fishy

@jekoorn

jekoorn commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Image there is something fishy

what do you mean?

@felixhekhorn

Copy link
Copy Markdown
Contributor

Image there is something fishy

what do you mean?

sorry - too much German I guess 🙈 the string(xir, xif = was found 12 times in the changed files - this is 4 times too much; it should be 3*3-1 = 8 times

@jekoorn

jekoorn commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

I added it also in the runcards for IHOU + MHOU missing prescription (that @Radonirinaunimi ) recommended be removed for future iterations.

@felixhekhorn

Copy link
Copy Markdown
Contributor

I added it also in the runcards for IHOU + MHOU missing prescription (that @Radonirinaunimi ) recommended be removed for future iterations.

I'm slightly confused ... what is the difference between 41_042_021 vs. 41_042_026 vs. 41_042_028? they are identical theory cards, but they differ by the FK table they contain? (@Radonirinaunimi I guess?)

(cc @scarlehoff this is again what we were discussing the other day I think)

@scarlehoff

Copy link
Copy Markdown
Member

If the theory cards are identical then the content should be identical (thus, they should not exist)

If there is any difference that cannot be reflected in the theory card (for instance that one uses yadism old version and the other yadism new version) that difference should be explicit in the description.

@jekoorn

jekoorn commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

I added it also in the runcards for IHOU + MHOU missing prescription (that @Radonirinaunimi ) recommended be removed for future iterations.

I'm slightly confused ... what is the difference between 41_042_021 vs. 41_042_026 vs. 41_042_028? they are identical theory cards, but they differ by the FK table they contain? (@Radonirinaunimi I guess?)

(cc @scarlehoff this is again what we were discussing the other day I think)

I did it like this because some other theories also did it like this; I assumed these would differ by FK-Table as you say, since potentially they would take DY/DIS tables from other theories (like is don ein some other theories like the 1000 series)

@scarlehoff

Copy link
Copy Markdown
Member

We are trying not to repeat the errors of the past :__)

Many of those were actually the same to the point that in the server they are linked among them.

@felixhekhorn

felixhekhorn commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

I did it like this because some other theories also did it like this; I assumed these would differ by FK-Table as you say, since potentially they would take DY/DIS tables from other theories (like is don ein some other theories like the 1000 series)

that is what we, @scarlehoff and I, concluded the other day (if I understood correctly): they differ by FK tables - and, no, such an information has no direct counterpart in the theory card, but I agree with

that difference should be explicit in the description.

so instead of a yaml comment in this specific file validphys2/src/validphys/scalevariations/scalevariationtheoryids.yaml, an additional ~ "As 40_005_049 but DIS from 40_005_027" in the description would be good

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.

4 participants