Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Commit 8f0975c

Browse files
authored
Add a discussion point about CompositeTensor.
1 parent 4fc8600 commit 8f0975c

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

rfcs/20200519-csr-sparse-matrix.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,8 +317,9 @@ Other sparse APIs follow NumPy and SciPy APIs. See links in [Supported Operation
317317
* Any comments on the APIs?
318318
* Are there any more operations that we should support?
319319
* Should we add `CSRSparseMatrix` support to existing standard ops as well, e.g., `tf.math.{add,asin,atan,ceil}`, etc?
320-
* Would love to hear more use cases.
320+
* Would love to hear about more use cases.
321321
* For neural networks, would CSR be useful for you (while Block CSR is still a future work)?
322+
* Should we make CSRSparseMatrix a [CompositeTensor](https://cs.opensource.google/tensorflow/tensorflow/+/v2.2.0-rc4:tensorflow/python/framework/composite_tensor.py;l=31)? Would the effort be worth it since we will transition to the new TFRT/MLIR backend soon? How should this be prioritized?
322323
* Should `SparseMatrix` replace `SparseTensor`?
323324

324325

0 commit comments

Comments
 (0)