-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
add PermMatrixCSC #78
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #78 +/- ##
==========================================
+ Coverage 88.85% 91.38% +2.52%
==========================================
Files 14 14
Lines 754 789 +35
==========================================
+ Hits 670 721 +51
+ Misses 84 68 -16 ☔ View full report in Codecov by Sentry. |
If it's non-breaking we should only bump patch version according to the convention. Don't bump the version inside PR. |
We should gradually switch to the column majored permutation matrix for better performance.
NOTE: this is a non-breaking feature.