#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):
ARRGH, I am so unbelievably stupid!! I forgot to apply my second patch!
All the timings for (3) above were only with the first patch!!!
On the bright side, it shows that my first patch isn't bad at all.
But I think it still makes sense to add the second as well. Here are the
timings with both patches.
__Creation of finite fields__
{{{
sage: def test():
....: for p in prime_range(10000):
....: P = GF(p)
....:
sage: %time test()
CPU times: user 0.68 s, sys: 0.02 s, total: 0.70 s
Wall time: 0.71 s
}}}
__Creation of polynomial rings__
{{{
sage: def test():
....: for p in prime_range(10000):
....: P = GF(p)['x']
....: P = GF(p)['x','y']
....:
sage: %time test()
CPU times: user 4.32 s, sys: 0.07 s, total: 4.39 s
Wall time: 4.40 s
}}}
'''__The Elliptic Curve Tests__'''
__prove_BSD()__
{{{
sage: %time L = EllipticCurve('960d1').prove_BSD()
CPU times: user 4.20 s, sys: 0.07 s, total: 4.27 s
Wall time: 4.46 s
}}}
__abelian_group()__
{{{
sage: def test():
....: E = EllipticCurve('389a')
....: for p in prime_range(10000):
....: if p != 389:
....: G = E.change_ring(GF(p)).abelian_group()
....:
sage: %time test()
CPU times: user 18.02 s, sys: 0.09 s, total: 18.11 s
Wall time: 18.16 s
}}}
In conclusion, the original problem has almost vanished.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11900#comment:66>
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.
