-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
large performance regression in obtaining sparse array from a transpose of a sparse array #28948
Labels
performance
Must go faster
regression
Regression in behavior compared to a previous version
sparse
Sparse arrays
Comments
Using |
Ah yes! I realized that literally just 2 minutes before you posted this, thanks! That's a huge relief, it might have been a lot of work to get around this. Ok, I'll try to make a PR if the fix is straight-forward enough. |
This was referenced Aug 29, 2018
nalimilan
added
the
regression
Regression in behavior compared to a previous version
label
Sep 3, 2018
Merged
KristofferC
pushed a commit
that referenced
this issue
Sep 4, 2018
KristofferC
pushed a commit
that referenced
this issue
Sep 4, 2018
KristofferC
pushed a commit
that referenced
this issue
Sep 8, 2018
KristofferC
pushed a commit
that referenced
this issue
Sep 8, 2018
Fixed by #28954 |
KristofferC
pushed a commit
that referenced
this issue
Feb 11, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
performance
Must go faster
regression
Regression in behavior compared to a previous version
sparse
Sparse arrays
On 0.6.4:
On 1.0.0
This scales very badly with the size of the array. This makes life extremely difficult if you want to get the "
I, J, V
" values of the resulting transpose.In the meantime any advice on a work around would be hugely appreciated.
The text was updated successfully, but these errors were encountered: