#6672: [with patch, positive review] Elliptic curve isogeny coercion of output
to
codomain curve takes too long
---------------------------------------+------------------------------------
Reporter: shumow | Owner: shumow
Type: defect | Status: new
Priority: minor | Milestone: sage-4.1.2
Component: elliptic curves | Keywords:
Reviewer: John Cremona, Alex Ghitza | Author: William Stein, Dan
Shumow, John Cremona
Merged: |
---------------------------------------+------------------------------------
Changes (by newvalueoldvalue):
* reviewer: => John Cremona, Alex Ghitza
* milestone: => sage-4.1.2
* author: => William Stein, Dan Shumow, John Cremona
Comment:
Positive review. Apply the three patches in order.
Tested on some random examples such as:
BEFORE THE PATCHES:
{{{
sage: E = EllipticCurve('109a').change_ring(GF(71))
sage: lis = E.rational_points()
sage: P = lis[20]
sage: timeit('E(P)')
625 loops, best of 3: 840 µs per loop
}}}
AFTER THE PATCHES:
{{{
sage: E = EllipticCurve('109a').change_ring(GF(71))
sage: lis = E.rational_points()
sage: P = lis[20]
sage: timeit('E(P)')
625 loops, best of 3: 191 µs per loop
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6672#comment:4>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
