-
Notifications
You must be signed in to change notification settings - Fork 10
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
[on hold] Support for sub arrays of dynamic size (jagged arrays) #305
base: develop
Are you sure you want to change the base?
Conversation
cc4b647
to
dd45ecc
Compare
bf0f7fc
to
641b014
Compare
60c4dca
to
7ab012e
Compare
7ab012e
to
d049713
Compare
* allow dynamic field types in the record dimension * add specializations to most of the core functions * add llama::dynamic to signal a dynamic array member in a RecordCoord * extend VirtualRecord to allow holding dynamic indices * extend blobNrAndOffset to allow for additional dynamic indices * add OffsetTable mapping * add customization allowing to dump OffsetTable mappings * add a few unit tests
d049713
to
2b4b65b
Compare
I am interested in this. Any ETA available? |
Hi! There is no ETA on this feature, since nobody is working on it. However, you can model jagged arrays yourself by creating replacing a dynamic sub array by an integral offset and creating a separate view for the dynamic sub array's structural type. This also corresponds to the implementation for jagged arrays in ROOT or Awkward Array. |
This PR adds support for fields of dynamic array type. Specifically: