Opened 7 years ago

Closed 7 years ago

#16336 closed enhancement (fixed)

Update Cremona's table of elliptic curves to 350000

Reported by: cremona Owned by:
Priority: minor Milestone: sage-6.3
Component: packages: optional Keywords: elliptic curve database
Cc: Merged in:
Authors: John Cremona Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: a725638 (Commits, GitHub, GitLab) Commit: a725638906f6395804d7cb13a1da8386b728d22d
Dependencies: Stopgaps:

Status badges

Description (last modified by cremona)

The new upstream tarball is at

http://boxen.math.washington.edu/home/cremona/database_cremona_ellcurve-20140512.tar.bz2

The new commit only changes the package-version.txt and checksums.ini files.

Change History (16)

comment:1 Changed 7 years ago by cremona

  • Summary changed from Update Cremona's table of elliptic curves to 340000 to Update Cremona's table of elliptic curves to 350000

comment:2 Changed 7 years ago by cremona

  • Description modified (diff)

comment:3 Changed 7 years ago by cremona

  • Description modified (diff)
  • Status changed from new to needs_review

comment:4 follow-up: Changed 7 years ago by jdemeyer

  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to positive_review

comment:5 in reply to: ↑ 4 Changed 7 years ago by cremona

Replying to jdemeyer:

Thanks! I am planning a pause now; the last batch of 10000 conductors took 7 weeks anyway.

comment:6 follow-up: Changed 7 years ago by vbraun

  • Status changed from positive_review to needs_work

There is no branch to merge...

comment:7 in reply to: ↑ 6 ; follow-up: Changed 7 years ago by cremona

Replying to vbraun:

There is no branch to merge...

There was when I left it for review! Where can it have gone? It was called u/cremona/ticket/16336 if I recall. Also, I assume tha Jeroen actually used the branch I uploaded...

comment:8 in reply to: ↑ 7 Changed 7 years ago by jdemeyer

Replying to cremona:

Also, I assume that Jeroen actually used the branch I uploaded...

Maybe not.

I simply did

./sage -dev checkout --ticket 16336

which makes a new branch if the Trac ticket has no branch.

comment:9 Changed 7 years ago by cremona

  • Branch set to u/cremona/ticket/16336

comment:10 Changed 7 years ago by cremona

  • Branch changed from u/cremona/ticket/16336 to u/cremona/trac/16336
  • Commit set to a725638906f6395804d7cb13a1da8386b728d22d

comment:11 Changed 7 years ago by cremona

I put it back. Perhaps I got the branch name mixed up, it really is u/cremona/trac/16336 .

comment:12 Changed 7 years ago by cremona

  • Status changed from needs_work to needs_review

JUst notices this had the fatal tag "needs work". I am putting it back to "needs review" but if the branch has not disappeared I think anyone can switch it back to "positive review".

comment:13 Changed 7 years ago by cremona

Please can someone review this (again)?

comment:14 follow-up: Changed 7 years ago by vbraun

  • Status changed from needs_review to positive_review

comment:15 in reply to: ↑ 14 Changed 7 years ago by cremona

Replying to vbraun: Thanks!

comment:16 Changed 7 years ago by vbraun

  • Branch changed from u/cremona/trac/16336 to a725638906f6395804d7cb13a1da8386b728d22d
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.