[ 
https://issues.apache.org/jira/browse/ARROW-6671?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16936779#comment-16936779
 ] 

Rok Mihevc commented on ARROW-6671:
-----------------------------------

SparseCSRMatrix might be more misleading as it 'doesn't look look like' a 
Tensor type. I think that is potentially more confusing than it being limited 
to 2D.

+1 for the consistent naming

> [C++] Sparse tensor naming
> --------------------------
>
>                 Key: ARROW-6671
>                 URL: https://issues.apache.org/jira/browse/ARROW-6671
>             Project: Apache Arrow
>          Issue Type: Wish
>          Components: C++
>            Reporter: Antoine Pitrou
>            Assignee: Kenta Murata
>            Priority: Minor
>
> Currently there's {{SparseCOOIndex}} and {{SparseCSRIndex}}, but also 
> {{SparseTensorCOO}} and {{SparseTensorCSR}}.
> For consistency, it would be nice to rename the latter {{SparseCOOTensor}} 
> and {{SparseCSRTensor}}.
> Also, it's not obvious the {{SparseMatrixCSR}} alias is useful.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to