Ticket #12617 (closed enhancement: fixed)

Opened 15 months ago

Last modified 14 months ago

Update Cremona's table of elliptic curves

Reported by: cremona Owned by: tbd
Priority: major Milestone: sage-5.0
Component: packages: optional Keywords: elliptic curve database, rd2, database
Cc: rohana Work issues:
Report Upstream: N/A Reviewers: John Cremona, Jennifer Balakrishnan
Authors: R. Andrew Ohana Merged in: sage-5.0.beta10
Dependencies: Stopgaps:

Description (last modified by cremona) (diff)

The optional spkg database_cremona_ellcurve-20120113.spkg needs to be updated to include the conductor range 220000-230000.

Updated spkg:  http://wstein.org/home/ohanar/cremona-database/database_cremona_ellcurve-20120302.spkg

Apply patch trac_12167-ec.patch

Attachments

trac_12167-ec.patch Download (920 bytes) - added by cremona 15 months ago.

Change History

comment:1 Changed 15 months ago by rohana

  • Status changed from new to needs_review
  • Description modified (diff)
  • Authors set to R. Andrew Ohana

comment:2 Changed 15 months ago by cremona

  • Reviewers set to John Cremona

Thanks -- testing now.

Changed 15 months ago by cremona

comment:3 Changed 15 months ago by cremona

  • Description modified (diff)

The patch fixes a problem testing sage/doc/en/bordeaux/elliptic_curves.rst caused by new curves in the database with generators different from those computed on the fly.

I give a positive review to the spkg; anyone who approves of my small patch can set the ticket to positive review.

comment:4 Changed 14 months ago by jen

  • Status changed from needs_review to positive_review
  • Reviewers changed from John Cremona to John Cremona, Jennifer Balakrishnan

The patch to the documentation looks good. Positive review.

comment:5 Changed 14 months ago by jen

  • Keywords database, rd2 added; database removed

comment:6 Changed 14 months ago by jen

  • Keywords rd2, database added; rd2 removed

comment:7 Changed 14 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-5.0.beta10
Note: See TracTickets for help on using tickets.