Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Cremona's table of elliptic curves to 240000 #13095

Closed
JohnCremona opened this issue Jun 8, 2012 · 10 comments
Closed

Update Cremona's table of elliptic curves to 240000 #13095

JohnCremona opened this issue Jun 8, 2012 · 10 comments

Comments

@JohnCremona
Copy link
Member

On 2012-06-06 John Cremona published an update of his database covering the conductor range 230000-240000. The optional spkg database_cremona_ellcurve needs updating accordgingly.


Updated spkg:
http://homepages.warwick.ac.uk/staff/J.E.Cremona/ftp/database_cremona_ellcurve-20120606.spkg

Apply patch attachment: trac13095-database-cremona.patch and then attachment: 13095_reviewer.patch

CC: @ohanar

Component: packages: optional

Keywords: elliptic curve database

Author: John Cremona

Reviewer: R. Andrew Ohana

Merged: sage-5.1.beta4

Issue created by migration from https://trac.sagemath.org/ticket/13095

@JohnCremona

This comment has been minimized.

@JohnCremona
Copy link
Member Author

Applies to 5.1.beta2

@JohnCremona
Copy link
Member Author

comment:2

Attachment: trac13095-database-cremona.patch.gz

@JohnCremona
Copy link
Member Author

comment:3

Andrew, I hope you can review this as it's the first time I did this myself -- and it was so incredibly easy (thanks!).

@ohanar

This comment has been minimized.

@ohanar
Copy link
Member

ohanar commented Jun 8, 2012

Reviewer: R. Andrew Ohana

@ohanar
Copy link
Member

ohanar commented Jun 8, 2012

comment:4

Attachment: 13095_reviewer.patch.gz

Just made a couple of small fixes in the doctests, otherwise everything looks good.

@JohnCremona
Copy link
Member Author

comment:5

Replying to @ohanar:

Just made a couple of small fixes in the doctests, otherwise everything looks good.

Thanks -- I did a full test but forgot the optional tag.

@haraldschilly
Copy link
Member

comment:7

spkg is on the server now

@jdemeyer
Copy link

Merged: sage-5.1.beta4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants