pytensor icon indicating copy to clipboard operation
pytensor copied to clipboard

`SparseTensorVariable` should not subclass `TensorVariable`

Open jessegrabowski opened this issue 1 month ago • 0 comments

Description

This came up in the context of #1674. We should consider reverting https://github.com/aesara-devs/aesara/issues/142. Sparse matrices and dense tensor are conceptually different enough that it doesn't make sense for one to be a direct subclass of the other.

I think there is room for a new abstract superclass that could define a common API they both need to adhere to, but that would be beyond the scope of what this issue is asking for.

jessegrabowski avatar Oct 18 '25 17:10 jessegrabowski