Opened 6 years ago

Closed 6 years ago

#1100 closed defect (fixed)

[with patch] polynomial roots() method can return rational roots for polynomials over ZZ

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

Description

According to the documentation, .roots() is only supposed to return values from the base ring, so this is a bug:

sage: x = polygen(ZZ)
sage: f = 2*x-3
sage: f.roots()
[(3/2, 1)]

Attachments (1)

1100.patch (2.0 KB) - added by cwitty 6 years ago.

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by cwitty

comment:1 Changed 6 years ago by cwitty

  • Summary changed from polynomial roots() method can return rational roots for polynomials over ZZ to [with patch] polynomial roots() method can return rational roots for polynomials over ZZ

This patch was developed after #995, and may not apply cleanly if the patch from #995 has not yet been applied.

comment:2 Changed 6 years ago by mabshoff

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

applied to 2.8.12.rc0

Note: See TracTickets for help on using tickets.