Opened 6 years ago

Closed 6 years ago

#1356 closed defect (fixed)

[with patch, with positive review] fix bug when taking abs() of exactly known QQbar

Reported by: cwitty Owned by: somebody
Priority: major Milestone: sage-2.8.15
Component: basic arithmetic Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

The following test fails in 2.8.15.alpha1:

            sage: v = QQbar.zeta(3) + 1
            sage: v.exactify()
            sage: v.abs().minpoly()

but the attached patch fixes it.

Attachments (1)

1356.patch (1.1 KB) - added by cwitty 6 years ago.

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by cwitty

comment:1 Changed 6 years ago by rlm

  • Summary changed from [with patch] fix bug when taking abs() of exactly known QQbar to [with patch, tested] fix bug when taking abs() of exactly known QQbar

Now:

sage: v = QQbar.zeta(3) + 1
sage: v.exactify()
sage: v.abs().minpoly()
x - 1

comment:2 Changed 6 years ago by rlm

  • Summary changed from [with patch, tested] fix bug when taking abs() of exactly known QQbar to [with patch, with positive review] fix bug when taking abs() of exactly known QQbar

comment:3 Changed 6 years ago by was

looks good to me.

comment:4 Changed 6 years ago by mabshoff

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

Merged in 2.8.15.rc0.

Note: See TracTickets for help on using tickets.