#17405: solve_right with matrix right hand side fails over RDF and CDF
-------------------------------------+-------------------------------------
Reporter: fredrik.johansson | Owner: peterwicksstringfield
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-6.9
Component: linear algebra | Resolution:
Keywords: | Merged in:
Authors: | Reviewers: Vincent Delecroix
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/peterwicksstringfield/17405_solve_right|
5dde1c0f88abf7ec28e4048da02e76bbbe410ff1
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by vdelecroix):
* reviewer: => Vincent Delecroix
* milestone: sage-6.5 => sage-6.9
Comment:
Hello,
1. You should not try to change the ring if it is already the good one as
it performs a copy of the matrix
{{{
sage: m = matrix(ZZ,3,4,range(12))
sage: m.change_ring(ZZ) is m
False
sage: %timeit m.change_ring(ZZ)
1000000 loops, best of 3: 988 ns per loop
sage: m = matrix(ZZ,10,10,range(100))
sage: %timeit m.change_ring(ZZ)
100000 loops, best of 3: 1.51 µs per loop
}}}
This is bad since linear system might involve huge matrices.
2. In your example, there is no need to print the input matrices `A` and
`B` (though it is not very bad).
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/17405#comment:5>
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.
