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.

Sign up for GitHub

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

Jump to bottom

Fix positivity names from 5 GeV to sqrt(5) GeV and DY -> Z0 #2011

Merged
merged 2 commits into from
Mar 19, 2024
Merged

Fix positivity names from 5 GeV to sqrt(5) GeV and DY -> Z0 #2011

merged 2 commits into from
Mar 19, 2024

Conversation

scarlehoff
Copy link
Member

No description provided.

@RoyStegeman
Copy link
Member

GEV2 is not supported?

@scarlehoff
Copy link
Member Author

No. And we already have the P syntax in a few places.

@scarlehoff scarlehoff added the run-fit-bot Starts fit bot from a PR. label Mar 18, 2024
Base automatically changed from nnpdf_data_package to master March 18, 2024 14:01
@github-actions GitHub Actions
Copy link

Greetings from your nice fit 🤖 !
I have good news for you, I just finished my tasks:

Check the report carefully, and please buy me a ☕ , or better, a GPU 😉!

@scarlehoff scarlehoff changed the title Fix positivity names from 5 GeV to sqrt(5) GeV Fix positivity names from 5 GeV to sqrt(5) GeV and DY -> Z0 Mar 18, 2024
@scarlehoff scarlehoff removed the run-fit-bot Starts fit bot from a PR. label Mar 18, 2024
@scarlehoff
Copy link
Member Author

I haven't found any other problems...

@RoyStegeman I might wait a few days in case I find more problems while removing the apfelcomb keys, and then I'll create a baseline with the new names

@RoyStegeman
Copy link
Member

@RoyStegeman I might wait a few days in case I find more problems while removing the apfelcomb keys, and then I'll create a baseline with the new names

Thanks, there is not really a hurry. The reason I asked for such a fit is that I generally modify the runcard of the most recent baseline fit when I write runcards for anything else. So having such a fit helps with making the new dataset names the standard in fits from now on.

@scarlehoff scarlehoff merged commit 8766dec into master Mar 19, 2024
6 checks passed
@scarlehoff scarlehoff deleted the positivity_fix_name branch March 19, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@scarlehoff @RoyStegeman