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

Reuse ekos for nFONLL #130

Closed
felixhekhorn opened this issue Oct 18, 2023 · 1 comment · Fixed by #153
Closed

Reuse ekos for nFONLL #130

felixhekhorn opened this issue Oct 18, 2023 · 1 comment · Fixed by #153
Labels
enhancement New feature or request refactor Refactor code

Comments

@felixhekhorn
Copy link
Contributor

FFNS and FFN0 are the same from the evolution point of view so there is no need to compute the same eko twice, we can just inherit

@felixhekhorn felixhekhorn added the refactor Refactor code label Oct 18, 2023
@giacomomagni giacomomagni added the enhancement New feature or request label Oct 19, 2023
@giacomomagni
Copy link
Contributor

giacomomagni commented Dec 7, 2023

Now that we have fixed the number of theory to be always 7, we should implement 2 commands doing:

The first one (nfonll_ekos):

  1. generating the 3 operator cards
  2. computing all the ekos in the 3 flavor patches
  3. inheriting the ekos

And a second one (nfonll_fks):

  1. computing the various fktables
  2. combining the fktables

Both command should work for multiple datasets.
This way the user will not have to loop manually on the nFONLL theories which is really error prone.

We can also think of the "one command" automation to get straight the fk-table, but that will not be useful for Sv, for which you don't want to recompute the eko each time...

@giacomomagni giacomomagni linked a pull request Feb 15, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor Refactor code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants