#14506: Echelonize leads to wrong multiplication
-------------------------------------+-------------------------------------
Reporter: mraum | Owner: jason, was
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-6.4
Component: linear algebra | Resolution:
Keywords: linear algebra, | Merged in:
echelonize, cache, mutation | Reviewers:
Authors: Darij Grinberg | Work issues:
Report Upstream: N/A | Commit:
Branch: | 5d90155e113f8ce3530568345d1b0415cf215cc6
public/linalg/echelonize_clear_cache| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Darij Grinberg', 'oldvalue': ''}):
* status: new => needs_review
* author: => Darij Grinberg
* cc: was (added)
* branch: => public/linalg/echelonize_clear_cache
* keywords: => linear algebra, echelonize, cache, mutation
* commit: => 5d90155e113f8ce3530568345d1b0415cf215cc6
Comment:
The mistake is in the fact that echelonizing a rational matrix in place
does not clear the `clear_denom` value of its cache. Fixed. Experiments
seem to suggest that other matrix types don't suffer from this bug, but
I'd rather see someone doublecheck this (I don't understand the really
low-level parts of the code).
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=5d90155e113f8ce3530568345d1b0415cf215cc6
5d90155]||{{{clear cache upon echelonization of rational dense
matrices}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/14506#comment:7>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
