#11900: Serious regression caused by #9138
---------------------------+------------------------------------------------
Reporter: SimonKing | Owner: tbd
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.3
Component: performance | Keywords: categories regression
Work_issues: | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies: #9138 #11911
---------------------------+------------------------------------------------
Comment(by SimonKing):
Note that with both patches applied, the creation of multivariate
polynomial rings is faster than the creation of univariate rings:
{{{
sage: L = [GF(p) for p in prime_range(10000)]
sage: def test():
....: for p in prime_range(10000):
....: P = GF(p)['x']
....:
sage: def test2():
....: for p in prime_range(10000):
....: P = GF(p)['x','y']
....:
sage: %time test()
CPU times: user 2.56 s, sys: 0.04 s, total: 2.60 s
Wall time: 2.61 s
sage: %time test2()
CPU times: user 1.22 s, sys: 0.02 s, total: 1.24 s
Wall time: 1.24 s
}}}
The computation time with sage-4.6.2 has only been 1.25 s for test() and
0.67 s for test2(). I doubt that a regression in the polynomial ring
creation can be avoided, when we really want category support for
polynomial rings. However, I'll do my best to improve it.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11900#comment:67>
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.
