#1918: Matrices that are printed are not aligned
------------------------------+---------------------------------------------
   Reporter:  jason           |       Owner:  was         
       Type:  enhancement     |      Status:  needs_review
   Priority:  major           |   Milestone:  sage-4.3    
  Component:  linear algebra  |    Keywords:              
Work_issues:                  |      Author:              
   Upstream:  N/A             |    Reviewer:              
     Merged:                  |  
------------------------------+---------------------------------------------
Changes (by hivert):

 * cc: hivert, nthiery (added)


Comment:

 Replying to [comment:6 wcauchois]:
 > I think this is ready for review! The only doctest failure now is from
 sage/interfaces/maxima.py, but I'm pretty sure that has nothing to do with
 this patch (I saw it in the main branch of 4.3.alpha0 as well).

 This is some very good piece of work ! However, I'm soory to say that but
 I think it is not general enough. In combinatorics, we have a lot of 2d
 objects (Ferrers diagram, Young tableau). Right now it is hardcoded in the
 patch that this hook only apply for a list of matrices. should'nt we add
 some plugin saying that a particular kind of objects are printed by a 2d
 diagram. Or maybe should we apply the hook allways ? Or for any objects
 whose __repr__ string contains a newline ?

 But maybe we should keep the for another patch.

 By the way and as a consequence, I don't think this has to do with linear
 algebra.

 Cheers,

 Florent

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