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
Echelonize with transformation=True oddness for sparse matrix #11558
Comments
comment:1
Volker, Unfortunate - but at least it is fixed now. Passes tests in the obvious places, I'll run long tests overnight. Comments, in decreasing order of merit.
I don't know, just seems inefficient to go entry-by-entry. Not critical, just a thought. Long weekend here in the US, but I'll stick with this. Rob |
Reviewer: Rob Beezer |
comment:2
It seems like |
Updated patch |
comment:3
Attachment: trac_11558_echelon_form_with_transformation.patch.gz I've added more documentation. Some matrix backends don't support |
comment:4
Applies, builds, passes obvious tests on 4.7.1.alpha3. And looks good. I'll run full tests overnight and then flip this to positive review. Rob |
comment:5
Passes all long tests on 4.7.1.alpha3, so positive review. Thanks for cleaning this up. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Merged: sage-4.7.2.alpha1 |
Appy:
Component: linear algebra
Author: Volker Braun
Reviewer: Rob Beezer
Merged: sage-4.7.2.alpha1
Issue created by migration from https://trac.sagemath.org/ticket/11558
The text was updated successfully, but these errors were encountered: