scikit-hep / uproot5

ROOT I/O in pure Python and NumPy.
https://uproot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
242 stars 76 forks source link

chore: update reading support for RNTuple v1 #1338

Closed ariostas closed 6 days ago

ariostas commented 1 week ago

This PR adds support for the first stable version of the RNTuple binary format (v1.0.0.0). It's mostly just rearranging some things and adjusting some constants. Tests will fail until https://github.com/scikit-hep/scikit-hep-testdata/pull/164 is merged and released. This is still a work in progress, but it seems to mostly work already.

Note that this will completely break support for any files produced before v1

ariostas commented 1 week ago

This is now ready for review. All the previous functionality works again, and I fixed a small bug. All the tests should pass after https://github.com/scikit-hep/scikit-hep-testdata/pull/164 is merged and released.

The remaining things that are not supported are:

Instead of waiting until I finish implementing the first two, I think it would be good to get this out so that people can try it out and we can get some feedback.

jpivarski commented 6 days ago

@all-contributors please add @ariostas for code

allcontributors[bot] commented 6 days ago

@jpivarski

I've put up a pull request to add @ariostas! :tada: