Support for A <: AbstractMatrix
miakramer opened this issue · 1 comments
miakramer commented
Is there any particular reason the interface for Tensor
requires A <: DenseMatrix
for the storage type? Does this package assume column-major indexing, for example? If not, would a PR removing that requirement and instead requiring A <: AbstractMatrix
be welcome?
Jutho commented
Almost all operations that you can do with Tensor(Map)
objects, such as contractions and linear algebra factorizations, are currently only supported for DenseMatrix
. Actually, in practice, typically only an implementation for Matrix
is available; the parametric type is there to support CuArray
in the (hopefully near) future. I thus don't quite see what you hope to gain by widening this to AbstractMatrix
?