Ticket #4390 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch; with positive review] elliptic curves: heegner_index command broken

Reported by: was Owned by: was
Priority: major Milestone: sage-3.2
Component: number theory Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description 

sage: EllipticCurve('675b').heegner_index(-11)
Traceback (most recent call last):
...
TypeError: regulator() got an unexpected keyword argument 'verbose'

Attachments

sage-4390.patch Download (1.3 KB) - added by was 5 years ago.

Change History

Changed 5 years ago by was

comment:1 Changed 5 years ago by was

  • Summary changed from elliptic curves: heegner_index command broken to [with patch; needs review] elliptic curves: heegner_index command broken

comment:2 Changed 5 years ago by craigcitro

  • Summary changed from [with patch; needs review] elliptic curves: heegner_index command broken to [with patch; with positive review] elliptic curves: heegner_index command broken

Looks good.

comment:3 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 3.2.alpha2

Note: See TracTickets for help on using tickets.