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.

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

mu0 scale is not updated correctly in the operator cards. #146

Closed
giacomomagni opened this issue Dec 15, 2023 · 4 comments · Fixed by #147
Closed

mu0 scale is not updated correctly in the operator cards. #146

giacomomagni opened this issue Dec 15, 2023 · 4 comments · Fixed by #147
Labels
bug Something isn't working

Comments

@giacomomagni
Copy link
Contributor

giacomomagni commented Dec 15, 2023

We have a potential problem in the function that writes the operator cards, if the Q0 scale provided in the theory card is not the same as the one defined as mu0 in the template.

In particular here:

def write_operator_card(pineappl_grid, default_card, card_path, tcard):

we should add something as:

operators_card["mu0"] = tcard["Q0"]

Thanks @toonhasenack and @Radonirinaunimi

@giacomomagni giacomomagni added the bug Something isn't working label Dec 15, 2023
@giacomomagni giacomomagni changed the title Q0 scale is not updated correctly mu0 scale is not updated correctly in the operator cards. Dec 15, 2023
@felixhekhorn
Copy link
Contributor

I agree, this is an oversight.

PS: remember that in NNPDF/eko#295 we also shift $n_{f,0}$ from theory to operator

@giacomomagni
Copy link
Contributor Author

giacomomagni commented Dec 15, 2023

PS: remember that in NNPDF/eko#295 we also shift nf,0 from theory to operator

okay thanks for the reminder. That one does not require an hot fix now, right?

@giacomomagni
Copy link
Contributor Author

I feel we have been lucky here...

@felixhekhorn
Copy link
Contributor

okay thanks for the reminder. That one do not require an hot fix now, right?

no that is still for the now v0.15 series and unmerged

I feel we have been lucky here...

I agree 🙈

felixhekhorn added a commit that referenced this issue Dec 15, 2023
@giacomomagni giacomomagni linked a pull request Dec 15, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants