Ticket #1356 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

[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: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
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

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

Change History

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

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

Merged in 2.8.15.rc0.

Note: See TracTickets for help on using tickets.