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

Allow to run explicitly a grid / fktable #168

Open
scarlehoff opened this issue Apr 9, 2024 · 2 comments
Open

Allow to run explicitly a grid / fktable #168

scarlehoff opened this issue Apr 9, 2024 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@scarlehoff
Copy link
Member

We are using now the nnpdf data and that's nice because you can define the datasets you want and get the fktables/grids automagically.

However, I think it would be nice to be able to say "I want all fktables I find doing ls grids/400/*". Specially for testing/debugging. Basically a --raw option that would bypass both the NNPDF autodiscovery and the yamldb.

@scarlehoff scarlehoff added the enhancement New feature or request label Apr 9, 2024
@felixhekhorn
Copy link
Contributor

felixhekhorn commented Apr 16, 2024

but we still have the "file-based"-mode in game - and it is even top level, which we were challenging at some point (but haven't taken any action yet)

$ poe pineko convolute --help
Poe => pineko convolute --help
Usage: pineko convolute [OPTIONS] GRID_PATH OP_PATH FKTABLE MAX_AS MAX_AL

  Convolute PineAPPL grid and EKO into an FK table.

  GRID_PATH and OP_PATH are the path to the respective elements to convolute,
  and FKTABLE is the path where to dump the output.

all plain file paths there (to be fair it is not sufficient to just spell the grid)

just to be specific the discussion was to rename pineko convolute -> pineko file convolute and pineko theory fks -> pineko fks (names are open for discussion)

@scarlehoff
Copy link
Member Author

My idea with this would be to go through the normal path (so generation of opcards and ekos and fonll if needed).

In practice (what I did it with bash before opening the issue) what I'd like is the same you would get if you "autogenerate" a <dataset>.yaml with FkTables: [[ <dataset> ]] as only content.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants