Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement string representation (__repr__) in sparse matrix classes #7848

Open
kmaehashi opened this issue Sep 4, 2023 · 1 comment · May be fixed by #7858
Open

Implement string representation (__repr__) in sparse matrix classes #7848

kmaehashi opened this issue Sep 4, 2023 · 1 comment · May be fixed by #7858
Labels
cat:enhancement Improvements to existing features contribution welcome good first issue

Comments

@kmaehashi
Copy link
Member

CuPy:

>>> cupyx.scipy.sparse.coo_matrix((10,10))
<cupyx.scipy.sparse._coo.coo_matrix object at 0x7fa5716da280>

SciPy:

>>> scipy.sparse.coo_matrix((10,10))
<10x10 sparse matrix of type '<class 'numpy.float64'>'
        with 0 stored elements in COOrdinate format>
@emcastillo emcastillo added cat:document Documentation good first issue cat:enhancement Improvements to existing features and removed cat:document Documentation labels Sep 5, 2023
@aadya940 aadya940 linked a pull request Sep 10, 2023 that will close this issue
Open
@rajarshighoshal
Copy link

I want to take the issue. Are we trying to make the repr exactly like that of scipy?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cat:enhancement Improvements to existing features contribution welcome good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add __repr__ in Sparse Matrix Classes aadya940/cupy
3 participants
@kmaehashi @emcastillo @rajarshighoshal