Opened 13 years ago

Closed 13 years ago

#4263 closed defect (fixed)

[with patch; positive review] elliptic curves -- point height serious stupid bug in raising error

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

Status badges

Description

This is dumb (first thing I tried broke):

sage: E = EllipticCurve('5077a1')
sage: F = E.change_ring(QuadraticField(-3,'a'))
sage: P = F([-2,3,1])
sage: s = P.height(); s
sage: type(s)
<type 'NoneType'>

Attachments (1)

sage-4263.patch (1.6 KB) - added by was 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by was

comment:1 Changed 13 years ago by was

  • Summary changed from elliptic curves -- point height serious stupid bug in raising error to [with patch; needs review] elliptic curves -- point height serious stupid bug in raising error

comment:2 Changed 13 years ago by malb

  • Summary changed from [with patch; needs review] elliptic curves -- point height serious stupid bug in raising error to [with patch; positive review] elliptic curves -- point height serious stupid bug in raising error

looks good to me.

comment:3 Changed 13 years ago by mabshoff

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

Merged in Sage 3.1.3.rc0

Note: See TracTickets for help on using tickets.