sprs icon indicating copy to clipboard operation
sprs copied to clipboard

CsMat type name

Open idmitrievsky opened this issue 10 years ago • 1 comments

Can you explain the naming? Why not just Mat type? I see that you use two formats CSR and CSC, so Cs prefix in type name doesn't clarify anything.

Am I missing something? If not, I would consider renaming to Mat. It can still be something like cs::Mat, but in that case users could actually use cs::Mat. It gives more choice to the user.

idmitrievsky avatar Sep 12 '15 05:09 idmitrievsky

That's because I'm planning to add block compressed (BSC/BSR) matrices later, and I was planning to name them BsMat. However it could be nice to have them as bs::Mat under your proposal.

I'll think about it but I'll probably do that. Thanks for the feedback!

vbarrielle avatar Sep 12 '15 13:09 vbarrielle