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.
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.
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
Enable nf=3 with
evolven3fit_new
. #1754Enable nf=3 with
evolven3fit_new
. #1754Changes from all commits
78c31b4
fe60a68
135862e
d63bda0
10dd7e2
6a7ef27
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand this work, so in principle is fine.
When you call
generate_q2grid
with both Q2s givennf0
is ignored. On contrary by default from the cli here we have Q2s to None so you actually want to usenf0
, it's just a bit confusing, but I don't see a quick fix as wellThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the
nf0
is acting a bit like a secret parameter to generate the q2grid from the defaults.I guess another option is not to call
generate_q2grid
when there is nothing to generate (just a default being used).There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe that's more clean.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to point out:
nf0
parameter is going to disappear, or better to move. Now that we are propagating evolution points from EKO, everything should be an evolution point (a point on a lane, rather than on the ladder), and also fitting scale will be an evolution point.So
nf0
should be always used, and neverNone
(despite being always 4, at least for the time being - but remember that is 3 for perturbative charm). You should be allowed to discard thenf
information only at the latest possible time (i.e. when generating the LHAPDF grid file).nf0
is an EKO extension to the NNPDF theory database, and inferred from the comparison ofQ0
withmc * kcThr
, but it would be better to always have it explicit, and infer as little as possible.