#13298 closed enhancement (fixed)
Update Cremona's table of elliptic curves to 270000
Reported by: | cremona | Owned by: | was |
---|---|---|---|
Priority: | minor | Milestone: | sage-5.4 |
Component: | packages: optional | Keywords: | Elliptic curves |
Cc: | ohanar | Merged in: | sage-5.4.beta1 |
Authors: | John Cremona | Reviewers: | R. Andrew Ohana |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
On 2012-08-27 John Cremona published an update of his database covering conductors up to 270000. The optional spkg database_cremona_ellcurve needs updating accordingly.
Updated spkg: http://homepages.warwick.ac.uk/staff/J.E.Cremona/ftp/database_cremona_ellcurve-20120827.spkg
Attachments (1)
Change History (20)
comment:1 Changed 8 years ago by
- Description modified (diff)
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
- Status changed from new to needs_review
comment:4 Changed 8 years ago by
Please fill in your real name as Author.
comment:5 Changed 8 years ago by
comment:6 Changed 8 years ago by
- Description modified (diff)
- Status changed from needs_review to needs_work
- Summary changed from Update Cremona's table of elliptic curves to 250000 to Update Cremona's table of elliptic curves to 270000
comment:7 Changed 8 years ago by
- Description modified (diff)
- Status changed from needs_work to needs_review
comment:8 Changed 8 years ago by
- Status changed from needs_review to needs_work
I'm now making a patch to allow for the issue raised above relating to #12768.
comment:9 Changed 8 years ago by
- Status changed from needs_work to needs_review
Without the patch there's one test which will fail if the optional database is installed since it relies on a curve of conductor 10001 not being in the database. I replaced it by one of conductor >10^8
.
comment:10 Changed 8 years ago by
- Cc ohanar added
comment:11 Changed 8 years ago by
- Reviewers set to R. Andrew Ohana
- Status changed from needs_review to positive_review
looks good
comment:12 Changed 8 years ago by
- Component changed from databases to optional packages
comment:13 Changed 8 years ago by
- Milestone changed from sage-5.3 to sage-5.4
comment:14 Changed 8 years ago by
spkg is on the sage server + on its way around the world :)
comment:15 Changed 8 years ago by
- Merged in set to sage-5.4.beta1
- Resolution set to fixed
- Status changed from positive_review to closed
comment:16 follow-up: ↓ 17 Changed 8 years ago by
Thanks! I was wondering if I would get to 280000 before this went in, but no (currently at 275840).
comment:17 in reply to: ↑ 16 ; follow-up: ↓ 18 Changed 8 years ago by
Replying to cremona:
Thanks! I was wondering if I would get to 280000 before this went in, but no (currently at 275840).
I now have an updated spkg which goes up to 280000. Shall I open a new ticket for that (I assume yes)?
comment:18 in reply to: ↑ 17 ; follow-up: ↓ 19 Changed 8 years ago by
Replying to cremona:
I now have an updated spkg which goes up to 280000. Shall I open a new ticket for that (I assume yes)?
Yes please! It's more work, but hard to follow later on, if one ticket that has been closed (and resolved something) ends up serving more than one task.
comment:19 in reply to: ↑ 18 Changed 8 years ago by
Replying to schilly:
Replying to cremona:
I now have an updated spkg which goes up to 280000. Shall I open a new ticket for that (I assume yes)?
Yes please! It's more work, but hard to follow later on, if one ticket that has been closed (and resolved something) ends up serving more than one task.
Fine. See #13547.
As soon as I have finished a full test to see if any doctests need changing (as sometimes happens) I'll set this to Needs Review.