#15036: Speed up matrix repr for matrices larger than 20
-------------------------------------+-------------------------------------
Reporter: nbruin | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: linear algebra | Resolution:
Keywords: | Merged in:
Authors: Nils Bruin | Reviewers: Travis Scrimshaw,
Report Upstream: N/A | Nathann Cohen
Branch: | Work issues:
public/ticket/matrix_repr_speedup-15036| Commit:
Dependencies: | f2d8c363fc26c1dfff9a868714cbc22bfc4759dc
| Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nbruin):
Replying to [comment:30 tscrim]:
> I've fixed the broken doctests that I found (`matrix_space.py` and the
`combinat/designs/latin_squares.py`). If someone could double-check, I
think it's positive review.
I don't think you based your branch on the branch from comment
[comment:27]. I had to move the
{{{
from sage.matrix.matrix import is_Matrix
}}}
to inside `simple_format_obj` because otherwise Sage won't start for me:
the IPython crash report has a long traceback, showing that somewhere
along the line, that import statement fails because the name `ZZ` isn't
available (yet).
This problem only arises with interactive use (and I guess in doctests
that test sage via an expect interface and hence running with IPython). It
would be surprising, but not impossible, if you're not experiencing this
problem (in which case import resolutions aren't deterministic), but the
fact that the problematic situation can arise we need to fix it. The
relevant (two line) change is at the bottom of
[http://git.sagemath.org/sage.git/commit/?id=d41816c57ede520353ed24ea9534c1219137d295].
--
Ticket URL: <http://trac.sagemath.org/ticket/15036#comment:31>
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.
