Opened 10 years ago

Closed 10 years ago

#12972 closed defect (fixed)

Fix small bug in elliptic curve saturation

Reported by: John Cremona Owned by: John Cremona
Priority: major Milestone: sage-5.4
Component: elliptic curves Keywords: saturation
Cc: Merged in: sage-5.4.beta1
Authors: John Cremona Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

The saturation() method for elliptic curves of Q has a small bug: a variable (which will most often notbe used) was not initialised. At the same time I changed two parameter in the saturation code.

Attachments (1)

Qsat.patch (1.4 KB) - added by John Cremona 10 years ago.
Applies t.o 5.0

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by John Cremona

Attachment: Qsat.patch added

Applies t.o 5.0

comment:1 Changed 10 years ago by John Cremona

Status: newneeds_review

I have been using this code (with the patch) for many thousands of curves over several months. I no longer remember which curve actually triggered the bug, so I have not added a doctest. WIth a new release I realised that I had never submitted the patch for review (but I will go on using it every day even if reviewers don't like the lack of doctest!).

comment:2 Changed 10 years ago by Volker Braun

Looks good to me!

comment:3 Changed 10 years ago by Volker Braun

Reviewers: Volker Braun
Status: needs_reviewpositive_review

comment:4 in reply to:  2 Changed 10 years ago by John Cremona

Replying to vbraun:

Looks good to me!

thanks!

comment:5 Changed 10 years ago by Jeroen Demeyer

Milestone: sage-5.3sage-5.4

comment:6 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.4.beta1
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.