#17405: solve_right with matrix right hand side fails over RDF and CDF
-------------------------------------+----------------------------
       Reporter:  fredrik.johansson  |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.5
      Component:  linear algebra     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
   Dependencies:                     |     Stopgaps:
-------------------------------------+----------------------------
Changes (by peterwicksstringfield):

 * status:  new => needs_review


Comment:

 I attached a patch to this ticket.

 Following the pattern set by the solve_right and solve_left in
 src/sage/matrix/matrix2.pyx, I tweaked the code in
 src/sage/matrix/matrix_double_dense.pyx to make solve_right and solve_left
 for RDF and CDF matrices accept matrices for b. If b is given as a matrix,
 the answer x will be returned as a matrix.

 Please note:
 1. The solve_left in matrix2 uses solve_right, to reduce redundancy; while
 the solve_left in matrix_double_dense does NOT use solve_right, and
 instead duplicates some code.
 2. The solve_left and solve_right in matrix_double_dense are now more
 lenient than those in matrix2, since they accept not just vectors and
 matrices for b, but also things that can be coerced into vectors (like
 Python lists).

 I didn't try to fix either of those issues.

--
Ticket URL: <http://trac.sagemath.org/ticket/17405#comment:1>
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.

Reply via email to