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

easy-to-fix mistake in the stein-watkins optional database docs #16313

Closed
williamstein opened this issue May 9, 2014 · 11 comments
Closed

easy-to-fix mistake in the stein-watkins optional database docs #16313

williamstein opened this issue May 9, 2014 · 11 comments

Comments

@williamstein
Copy link
Contributor

Hi William,

I just downloaded the sage version of the Stein-Watkins database of (a subset of) the elliptic curves of conductor < 10^8, because I need a supply of small curves with large rational torsion subgroups.  I looked in

  sage/local/share/stein_watkins/format.txt

and it says that the torsion subgroup is given as follows:

  torsion_subgroup - n or nx, where n means Z/nZ and nx means Z/2Z x Z/2nZ.

But in the actual tables, nx seems to indicate Z/2Z x Z/nZ (instead of 2nZ), because the values that appear are 2x, 4x, 6x, and 8x.

I hope all is well with you!  Thanks ---

-- Everett

We simply need the updated format.txt from http://modular.math.washington.edu/ecdb/

New upstream package: http://boxen.math.washington.edu/home/jdemeyer/spkg/database_stein_watkins-20110713.tar.gz

Upstream: Fixed upstream, but not in a stable release.

CC: @williamstein

Component: packages: huge

Author: Jeroen Demeyer

Branch/Commit: 2ae0cda

Reviewer: Volker Braun

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

@williamstein williamstein added this to the sage-6.2 milestone May 9, 2014
@nexttime nexttime mannequin modified the milestones: sage-6.2, sage-6.3 May 11, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@jdemeyer
Copy link

Upstream: Completely fixed; Fix reported upstream

@jdemeyer
Copy link

comment:3

This is fixed upstream (http://modular.math.washington.edu/ecdb/format.txt), so we should add the right format.txt in the package.

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link

Author: Jeroen Demeyer

@jdemeyer
Copy link

Branch: u/jdemeyer/ticket/16313

@jdemeyer
Copy link

New commits:

2ae0cdaUpdate format.txt in Stein--Watkins database

@jdemeyer
Copy link

Commit: 2ae0cda

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link

jdemeyer commented Sep 2, 2014

Changed upstream from Completely fixed; Fix reported upstream to Fixed upstream, but not in a stable release.

@vbraun
Copy link
Member

vbraun commented Oct 26, 2014

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Oct 26, 2014

Changed branch from u/jdemeyer/ticket/16313 to 2ae0cda

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

3 participants