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

Binary FKTables #23

Closed
scarrazza opened this issue Feb 8, 2018 · 3 comments
Closed

Binary FKTables #23

scarrazza opened this issue Feb 8, 2018 · 3 comments

Comments

@scarrazza
Copy link
Member

Issue by Zaharid
Friday Jun 23, 2017 at 16:20 GMT
Originally opened as https://github.com/NNPDF/libnnpdf/issues/11


The alphas fits are becoming unbearable due to the slowness of the fktable processing. As a first step can we simply compress each fktable individually? That's already quite a speed up, independent of everything else (and should be working now, right?).

For the binary, I think looking at something like avro is better than inventing the format ourselves:

https://avro.apache.org/docs/current/spec.html

in my opinion it has the things we need (and not many we don't).

@nhartland nhartland changed the title Binary compressed FKTables Binary FKTables Feb 20, 2018
@felixhekhorn
Copy link
Contributor

I guess PineAPPL grids are binary, so we can close - do you agree? @cschwan @scarrazza @Zaharid

@cschwan
Copy link
Contributor

cschwan commented Sep 7, 2022

Yes, they are binary; more concretely the PineAPPL files are

As for the size, have a at look NNPDF/pineappl#45 (comment) in which I discuss the conversion of the complete https://github.com/NNPDF/applgrids repository. For (PineAPPL-)FK tables I would expect a similar picture.

Footnotes

  1. the file header allows us to completely change all data structures and serialization formats in newer version if we decide we need that

  2. transparently means that you can run lz4 -d grid.pineappl.lz4 to decompress the file, which will get you the header + serde/bincode output. The decompressed output can also be read by every API just as well as the compressed file

@RosalynLP RosalynLP removed their assignment Sep 9, 2022
@RosalynLP
Copy link
Contributor

:)

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

4 participants