Documentation error for Stein-Watkins database

[sagetrac-kimball]

The documentation for one of the Stein-Watkins database functions appears to have a typo. Specifically, the current documentation here:

http://doc.sagemath.org/html/en/reference/databases/sage/databases/stein_watkins.html

says

The command SteinWatkinsPrimeData(n) returns an iterator
over the curves in the nth Stein-Watkins prime table, which
contains prime conductor elliptic curves of conductor between
n 10⁶ and (n+1) 10⁶.

Unless the function or the database has changed recently,
I believe the 10⁶ should be 10⁸. At least on my home
installation of Sage 8.7, when I run SteinWatkinsPrimeData(0),
I get conductors up to 10⁸.

CC: @slel @JohnCremona

Component: documentation

Keywords: Stein-Watkins database

Author: Frédéric Chapoton

Branch/Commit: 0ee4555

Reviewer: John Cremona

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

[embray]

comment:1

Ticket retargeted after milestone closed

[mkoeppe]

comment:2

Moving tickets to milestone sage-9.2 based on a review of last modification date, branch status, and severity.

[mkoeppe]

comment:5

Moving this ticket to 9.4, as it seems unlikely that it will be merged in 9.3, which is in the release candidate stage

[JohnCremona]

comment:7

In the raw data files, the documentation of files p.n says:

  p.n   -- curves between n*10^8 and (n+1)*10^8 with prime conductor

so it looks as if the suggested correction is good.

By the way, the Stein-Watkins database does not contain all curves in the range, just the ones they found using a search which was not intended to give complete lists. Since then, Bennett, Gherga and Rechnitzer (see https://www.adelagherga.ca/research/BeGhRe3.pdf) have a list of all curves of prime conductor up to 10¹⁰. So it would be possible to make an optional package from that data too (or instead). For primes up to 310⁸ I have computed all the additional data as for my own curve collection, and those are in the LMFDB -- but 10¹⁰ is a lot larger than 310⁸.

[mkoeppe]

comment:8

Setting a new milestone for this ticket based on a cursory review.

[mkoeppe]

comment:9

Stalled in needs_review or needs_info; likely won't make it into Sage 9.5.

[fchapoton]

Author: Frédéric Chapoton

[fchapoton]

Commit: 9d01ad7

[fchapoton]

comment:11

should be very easy to review

---

New commits:

9d01ad7

about Stein-Watlkins database

[fchapoton]

Branch: u/chapoton/28598

[sagetrac-git]

Changed commit from 9d01ad7 to 0ee4555

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

0ee4555

about Stein-Watkins database

[fchapoton]

comment:13

green bot, so please review

[mkoeppe]

Reviewer: John Cremona

[vbraun]

Changed branch from u/chapoton/28598 to 0ee4555