#15614: Update Cremona's table of elliptic curves to 320000
-------------------------------------+-------------------------------------
       Reporter:  cremona            |        Owner:  cremona
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.1
      Component:  packages:          |   Resolution:
  optional                           |    Merged in:
       Keywords:  elliptic curve     |    Reviewers:  Jeroen Demeyer
  database                           |  Work issues:
        Authors:  John Cremona       |       Commit:
Report Upstream:  N/A                |  6f0430bd73c0cd8bec0764c7b7b706cc4963a4c4
         Branch:                     |     Stopgaps:
  u/jdemeyer/ticket/15614            |
   Dependencies:  #15556             |
-------------------------------------+-------------------------------------

Comment (by cremona):

 Replying to [comment:22 jdemeyer]:
 > John, I am preparing a small reviewer patch (mostly fixing some
 documentation).
 >
 > However, I noticed the following in `src/sage/databases/cremona.py`:
 > {{{
 >         EXAMPLES::
 >
 >             sage: c = CremonaDatabase()
 >             sage: c.elliptic_curve_from_ainvs([0, -1, 1, -10, -20])
 >             Elliptic Curve defined by y^2 + y = x^3 - x^2 - 10*x - 20
 over Rational Field
 >             sage: c.elliptic_curve_from_ainvs([1, 0, 0, -101, 382])  #
 optional - database_cremona_ellcurve
 >             Elliptic Curve defined by y^2 + x*y = x^3 - 101*x + 382 over
 Rational Field
 >
 >         Old (pre-2006) Cremona labels are not allowed::
 >
 >             sage: c.elliptic_curve('9450KKKK1')
 >             Elliptic Curve defined by y^2 + x*y + y = x^3 - x^2 - 5*x +
 7 over Rational Field
 > }}}
 > Is the "not allowed" simply a typo for "allowed" or is something deeper
 going on?

 It's a typo.  I think it used to be true that the old labels were not
 allowed and this test showed an error.  It was probably me who changed
 that to a working test but slipped up.

 I approve of your other changes, so if you would remove "not" we can
 proceed.  Thanks!

--
Ticket URL: <http://trac.sagemath.org/ticket/15614#comment:25>
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/groups/opt_out.

Reply via email to