#10944: Check for similar matrices
-------------------------------+--------------------------------------------
    Reporter:  rbeezer         |         Owner:  jason, was       
        Type:  enhancement     |        Status:  closed           
    Priority:  minor           |     Milestone:  sage-4.7.1       
   Component:  linear algebra  |    Resolution:  fixed            
    Keywords:                  |   Work_issues:                   
    Upstream:  N/A             |      Reviewer:  Dan Drake        
      Author:  Rob Beezer      |        Merged:  sage-4.7.1.alpha2
Dependencies:                  |  
-------------------------------+--------------------------------------------

Comment(by rbeezer):

 Replying to [comment:9 kini]:
 > Someone on IRC (`Alligadi`) voiced concerns about lines 7417 and 7422 of
 sage/matrix/matrix2.pyx as added in the patch, namely that there are call
 parens missing from 7417 and that the second `self` should be `other` in
 7422. Typos?

 Correct.  Both are small mistakes.  No harm done, really, since the only
 consequence is the failure to avoid unnecessary computation.  In other
 words, results will be correct as written, but intended efficiencies will
 not happen.  I'll put up a patch tomorrow or so.

 Thanks for passing along the IRC intel.

 Rob

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10944#comment:10>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to