Opened 8 years ago

Closed 7 years ago

#16313 closed defect (fixed)

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

Reported by: was Owned by:
Priority: trivial Milestone: sage-6.4
Component: packages: huge Keywords:
Cc: was Merged in:
Authors: Jeroen Demeyer Reviewers: Volker Braun
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: 2ae0cda (Commits, GitHub, GitLab) Commit: 2ae0cdae93b928cb227f2ada7538efbad5d417f8
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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

Change History (9)

comment:1 Changed 8 years ago by leif

  • Milestone changed from sage-6.2 to sage-6.3

comment:2 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:3 Changed 7 years ago by jdemeyer

  • Report Upstream changed from N/A to Completely fixed; Fix reported upstream

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

comment:4 Changed 7 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Description modified (diff)

comment:5 Changed 7 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/16313
  • Created changed from 05/09/14 17:19:31 to 05/09/14 17:19:31
  • Modified changed from 08/19/14 19:37:20 to 08/19/14 19:37:20

comment:6 Changed 7 years ago by jdemeyer

  • Commit set to 2ae0cdae93b928cb227f2ada7538efbad5d417f8
  • Status changed from new to needs_review

New commits:

2ae0cdaUpdate format.txt in Stein--Watkins database

comment:7 Changed 7 years ago by jdemeyer

  • Cc was added
  • Description modified (diff)
  • Report Upstream changed from Completely fixed; Fix reported upstream to Fixed upstream, but not in a stable release.

comment:8 Changed 7 years ago by vbraun

  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

comment:9 Changed 7 years ago by vbraun

  • Branch changed from u/jdemeyer/ticket/16313 to 2ae0cdae93b928cb227f2ada7538efbad5d417f8
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.