-
Notifications
You must be signed in to change notification settings - Fork 4
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
A proposal for N-Dimensional sparse arrays #20
Conversation
Automated Review URLs |
I've added an example parser in https://github.com/willow-ahrens/Finch.jl/blob/0176e14e7731fa9062ff8e5638ea7fb5b55e9aa9/src/fileio/binsparse.jl |
One note: I included dense[n] because it makes things easy to explain, but I really think that because dense[1], dense[1] is equal to dense[2], we should only support dense[1] to keep things normalized. |
Another note: Finch understands things recursively, so that's how I've written the spec and the parser, but I get the sense that people may have been hoping for a list of format descriptors (i.e. |
I see--hierarchical is especially nice when a level spans multiple dimensions (such as COO). This may also be useful for e.g. DIA so there is less concern for how to compose and transform "1-d" levels to get more advanced levels. |
Here's an example of a property graph for CSC with an iso int capacity and a float weight
|
# Please enter the commit message for your changes. Lines starting
I think that sounds like a fine idea. Ultimately we all want it to live in one spec document, and we'll have to have some way of specifying what's v1 and what's a v2 extension. What you suggested sounds good to me. |
Alright, I'm merging this with language saying it's v2.0 only and subject to further discussion |
Here's my proposal for n-dimensional arrays in binsparse, to advocate for Erik's
sparse[n]
anddense[n]
approach. I have tried to keep things as close to the existing standard as possible, so the array names have stayed exactly the same as the proposed 1.0.I had to move iso from the value type to the format descriptor.