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

Add COO.nonzero method. #148

Merged
merged 1 commit into from May 3, 2018

Conversation

Projects
None yet
3 participants
@hameerabbasi
Collaborator

hameerabbasi commented May 3, 2018

Closes #145

@hameerabbasi hameerabbasi requested a review from mrocklin May 3, 2018

@codecov-io

This comment has been minimized.

codecov-io commented May 3, 2018

Codecov Report

Merging #148 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   95.81%   95.82%   +<.01%     
==========================================
  Files          10       10              
  Lines        1195     1197       +2     
==========================================
+ Hits         1145     1147       +2     
  Misses         50       50
Impacted Files Coverage Δ
sparse/coo/core.py 93.71% <100%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f2a9ae...22cc851. Read the comment docs.

@mrocklin mrocklin merged commit 3a53e87 into pydata:master May 3, 2018

4 checks passed

ci/circleci: build_27 Your tests passed on CircleCI!
Details
ci/circleci: build_36 Your tests passed on CircleCI!
Details
codecov/patch 100% of diff hit (target 95.81%)
Details
codecov/project 95.82% (+<.01%) compared to 8f2a9ae
Details
@mrocklin

This comment has been minimized.

Collaborator

mrocklin commented May 3, 2018

Nice to see this come out so easily

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment