search

metagraph-dev / mlir-graphblas

MLIR tools and dialect for GraphBLAS
https://mlir-graphblas.readthedocs.io/en/latest/
Apache License 2.0
16 stars 6 forks source link

Replace "#SparseVec64" with "CV64" #188

Closed paul-tqh-nguyen closed 3 years ago

paul-tqh-nguyen commented 3 years ago

In several places throughout our code base (e.g. in tersify_mlir) and documentation, we have two names for sparse vector encodings, #SparseVec64 and #CV64.

#SparseVec64 = #sparse_tensor.encoding<{
  dimLevelType = [ "compressed" ],
  pointerBitWidth = 64,
  indexBitWidth = 64
}>

We should be consistent and pick one. I vote for #CV64.

jim22k commented 3 years ago

+1 for #CV64

paul-tqh-nguyen commented 3 years ago

https://github.com/metagraph-dev/mlir-graphblas/pull/191 replaces #SparseVec64 with #CV64.