GraphBLAS / binsparse-specification

A cross-platform binary storage format for sparse data, particularly sparse matrices.
https://graphblas.org/binsparse-specification/
BSD 3-Clause "New" or "Revised" License
16 stars 4 forks source link

Represent ISO matrices using the type of the `values` array #28

Closed BenBrock closed 1 year ago

BenBrock commented 1 year ago

As discussed in our last meeting, update the spec to specify ISO matrices using the type of the values array instead of in the matrix format.

github-actions[bot] commented 1 year ago

Automated Review URLs

willow-ahrens commented 1 year ago

Looks good to me! I merged it into the types PR

willow-ahrens commented 1 year ago

fixes https://github.com/GraphBLAS/binsparse-specification/issues/26

willow-ahrens commented 1 year ago

@eriknw @jim22k We may want to also merge this one since we have consensus

BenBrock commented 1 year ago

I am okay to merge, if we have consensus.