#8814: Remove redundant checks for elliptic curve group structure
-------------------------------+--------------------------------------------
   Reporter:  robertwb         |       Owner:  cremona     
       Type:  enhancement      |      Status:  needs_review
   Priority:  major            |   Milestone:  sage-4.4.1  
  Component:  elliptic curves  |    Keywords:              
     Author:                   |    Upstream:  N/A         
   Reviewer:                   |      Merged:              
Work_issues:                   |  
-------------------------------+--------------------------------------------
Changes (by robertwb):

  * status:  new => needs_review


Comment:

 Before:

 {{{
 sage: F.<a>=GF(101^3,'a')
 sage: timeit("EllipticCurve([2*a^2 + 48*a + 27, 89*a^2 + 76*a +
 24]).abelian_group()")
 5 loops, best of 3: 1.37 s per loop
 sage: timeit("EllipticCurve(GF(1009), [2, 1]).abelian_group()")
 25 loops, best of 3: 21.1 ms per loop
 }}}

 After:

 {{{
 sage: F.<a>=GF(101^3,'a')
 sage: timeit("EllipticCurve([2*a^2 + 48*a + 27, 89*a^2 + 76*a +
 24]).abelian_group()")
 5 loops, best of 3: 1.28 s per loop
 sage: timeit("EllipticCurve(GF(1009), [2, 1]).abelian_group()")
 5 loops, best of 3: 15.2 ms per loop
 }}}

 Of course, this is just low hanging fruit (I've gotten 40x or more speedup
 in EC point arithmetic alone via Cython) but every little bit helps.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8814#comment:1>
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.

Reply via email to