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

Expose fastNLO converter to Python/Rust/... #128

Open
2 tasks
alecandido opened this issue Mar 16, 2022 · 5 comments
Open
2 tasks

Expose fastNLO converter to Python/Rust/... #128

alecandido opened this issue Mar 16, 2022 · 5 comments
Assignees

Comments

@alecandido
Copy link
Member

  • move the current fastnlo module from the CLI to the main crate
  • bind in the Python interface
@cschwan cschwan changed the title Expose fastNLO converter to Python Expose fastNLO converter to Python/Rust/... Mar 16, 2022
@cschwan cschwan self-assigned this Mar 16, 2022
@cschwan cschwan added this to the v0.5.2 milestone Mar 16, 2022
@cschwan
Copy link
Contributor

cschwan commented Mar 16, 2022

I'll take care of the first part, and you can write bindings afterwards!

@cschwan cschwan modified the milestones: v0.5.2, v0.5.3 Mar 29, 2022
@cschwan
Copy link
Contributor

cschwan commented Apr 13, 2022

I tried doing this and unfortunately didn't succeed, probably due to a linking-order problem with LHAPDF: it seems to matter whether I link LHAPDF first and fastNLO second, because the latter depends on the former and the CLI depends on both. I think that this problem can't be solved with cargo, so this Issue probably has to be postponed for a while.

@alecandido
Copy link
Member Author

Sad, but fine.

In any case, it is not a major issue at the moment: it was mainly needed to start writing "pinecards" for conversion, in order to have consistent metadata, include versions and so on.
But pinecards (I'd really like to call them seeds :D) are going to be reworked anyhow (at least a bit), after the new commondata will be available.

In the meantime, if we'll generate our own grids, we might stop doing systematic conversion. Let's worry afterwards.

@felixhekhorn
Copy link
Contributor

For writing the APPL conversion pinecards we need to run the "CLI" (meaning the appl2pine command) in any case (for the moment) so we can even call the PineAPPL CLI there ...

In any case NNPDF/pinefarm#3 is not liked to CommonData ...

@alecandido
Copy link
Member Author

I'd still assign low priority to this task: we already converted all the grids we needed, it's non-blocking for anything.
It's a refinement we want, but still a refinement.

@cschwan cschwan removed this from the v0.5.3 milestone Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants