Skip to content

[ENH] statistics: Speed up countnans for sparse matrices#2965

Merged
ajdapretnar merged 1 commit intobiolab:masterfrom
ales-erjavec:stats-sparse-na
Mar 23, 2018
Merged

[ENH] statistics: Speed up countnans for sparse matrices#2965
ajdapretnar merged 1 commit intobiolab:masterfrom
ales-erjavec:stats-sparse-na

Conversation

@ales-erjavec
Copy link
Contributor

Issue

Alleviates gh-2925

Description of changes
  • Optimize countans for sparse matrices
Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar
Copy link
Contributor

Thank you, thank you, thank you! 🙏

@codecov-io
Copy link

codecov-io commented Mar 22, 2018

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2965      +/-   ##
==========================================
+ Coverage   81.86%   81.86%   +<.01%     
==========================================
  Files         329      329              
  Lines       56785    56787       +2     
==========================================
+ Hits        46486    46488       +2     
  Misses      10299    10299

@lanzagar lanzagar added this to the 3.12 milestone Mar 23, 2018
@ajdapretnar
Copy link
Contributor

I was wondering would it make sense to turn off Remove unused features and Remove unused classes for sparse matrices? When you connect the data, this is on by default and it makes things a bit slow.

@ajdapretnar ajdapretnar self-assigned this Mar 23, 2018
@ajdapretnar ajdapretnar merged commit 37c639a into biolab:master Mar 23, 2018
@ales-erjavec ales-erjavec deleted the stats-sparse-na branch March 23, 2018 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants