-
Notifications
You must be signed in to change notification settings - Fork 63
[WIP] Add support for ragged arrays
#1104
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
base: main
Are you sure you want to change the base?
Conversation
|
Awesome! It looks like you found all the modules that need to be touched to add this. The aspect that will need the most careful thought is the structure description and the HTTP APIs. These are designed to be used not only from the built-in Python client, but also from The Awkward If we were willing to similarly restrict This A class RaggedStructure(ArrayStructure):
shape: Tuple[None | int, ...] # override base class which has this as Tuple[int, ...]I'm not sure whether tiled/tiled/structures/sparse.py Lines 19 to 23 in f6a9509
Although reusing the awkward form keeps things simple assuming your client already consumes awkward I think having a custom, much more constrained structure JSON, is worthwhile, to make ragged arrays a more portable and accessible concept. |
This adds client and backend support for reading/writing irregular arrays using the the
raggedpackage. Asraggedis more or less a wrapper aroundawkward, this PR reuses, or adds similar implementations from that structure family (e.g. serialization).Implements #801.
Checklist
[None]shaped data from Bluesky/TiledWriter into SQL storageRaggedAdapterreturned bySQLAdapter(SQL storage)