Use nf to generate alpha_s #391
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
felixhekhorn
requested changes
Jul 16, 2024
|
PS: it would have been more convenient to discuss here instead of the commit 🙈 |
I guess this is fine unless you give a wrapper that does the full thing. But given that the info file generation is its own thing it makes sense to allow for extra flexibility (sorry for commenting in the commit, I just followed the link in the email and didn't pay attention :___ ) |
|
I just rerun this after your changes (with the same eko from before) and seems to work the same |
felixhekhorn
approved these changes
Jul 18, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The information about nf is not utilized when generating the info files. This PR fixes that.
evolven3fit relies on eko for the info file so once this is merged into a release of eko it should work fine.
Apparently LHAPDF doesn't have a block mechanism for
alpha_sso at the moment we just provide two values for two nfs for the same Q at the thresholds.(As far as I can see, the same problem was there in NNPDF4.0 -apfel + evolven3fit- and NNPDF3.1 -apfel + evolvennfit-, so nobody ever noticed...)