#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: |
------------------------------+---------------------------------------------
Comment(by hivert):
Replying to [comment:10 wcauchois]:
> In one of my earlier iterations of the displayhook, I applied the
special list formatting to every "tall" list. But when I ran testall I
found that some objects just didn't work well with the formatting. So I
think that applying the hook always is a mistake, and besides would
introduce many more doctest headaches. Having objects "opt-in" by
including a special field or method that tells the displayhook to use the
special list formatting sounds like a good idea though.
I even tried to apply it on **all** lists ! I can tell you that there are
much more output that looks better with your modification than without.
(see e.g. in {{{sage/modular/arithgroup/}}}... I'm not sure doing it
systematically is a mistake... But some objects have to adapt their
output, and the way you decide to break a line or not is not always
optimal (see e.g. the output of {{{
TabularUnifiedsage/modular/modsym/relation_matrix.py}}} in your patch.
> This kind of work should definitely be done, but in another ticket. We
will have to come to an agreement about these design issues. For now,
let's get this patch into Sage.
Agreed, this show that they are a lot of possible improvement in sage
output routine. Thanks a lot for showing us how to do that.
I'm waiting for the results of the tests to give you a positive review...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/1918#comment:11>
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.
