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.
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
Getitem for major dimension #301
Conversation
734811b
to
083597e
Compare
LGTM, will be merged after #240! |
083597e
to
4e75e92
Compare
I fixed behavior of getitem for slices with negative values. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except a minor comment!
cupy/sparse/compressed.py
Outdated
major_stop = major_size | ||
if major_step is None: | ||
major_step = 1 | ||
print(major_start, major_stop, major_size) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this line.
Please resolve the conflicts. |
9099404
to
906e241
Compare
I rebased master. |
I will merge this after CI passes. |
I implemented getitem for major dimension of compressed matrixes.
Please merge #240 first.
related to #36