#10424: Expand matrix augment'ing to allow vectors as input
------------------------------+---------------------------------------------
   Reporter:  rbeezer         |       Owner:  jason, was  
       Type:  enhancement     |      Status:  needs_review
   Priority:  minor           |   Milestone:  sage-4.6.1  
  Component:  linear algebra  |    Keywords:              
     Author:  Rob Beezer      |    Upstream:  N/A         
   Reviewer:                  |      Merged:              
Work_issues:                  |  
------------------------------+---------------------------------------------

Comment(by jvkersch):

 I played with the patch, everything works as expected, HTML doesn't
 generate any warnings (and the docstrings are very clear and to the point
 -- a pleasure to read).  I'm currently running `make ptestlong`, and I can
 give the patch a positive review once that finishes.

 I agree that it would be more natural to coerce the elements of `self` and
 `other` into some common base ring -- this is the behavior that I
 intuitively expected, but I didn't see the "future improvements"
 description on this patch.

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