Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

polynomial roots() method can return rational roots for polynomials over ZZ #1100

Closed
sagetrac-cwitty mannequin opened this issue Nov 4, 2007 · 3 comments
Closed

polynomial roots() method can return rational roots for polynomials over ZZ #1100

sagetrac-cwitty mannequin opened this issue Nov 4, 2007 · 3 comments
Assignees
@sagetrac-cwitty
Labels
c: basic arithmetic p: major / 3 t: bug
Milestone
sage-2.8.12

Comments

@sagetrac-cwitty
Copy link
Mannequin

sagetrac-cwitty mannequin commented Nov 4, 2007

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)]

Component: basic arithmetic

Issue created by migration from https://trac.sagemath.org/ticket/1100
@sagetrac-cwitty sagetrac-cwitty mannequin added this to the sage-2.8.12 milestone Nov 4, 2007
@sagetrac-cwitty sagetrac-cwitty mannequin self-assigned this Nov 4, 2007
@sagetrac-cwitty
Copy link
Mannequin Author

sagetrac-cwitty mannequin commented Nov 4, 2007

Attachment: 1100.patch.gz

@sagetrac-cwitty
Copy link
Mannequin Author

sagetrac-cwitty mannequin commented Nov 4, 2007

comment:1

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

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Nov 6, 2007

comment:2

applied to 2.8.12.rc0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Nov 6, 2007
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-cwitty sagetrac-cwitty

Labels
c: basic arithmetic p: major / 3 t: bug
Projects
None yet
Milestone
sage-2.8.12
Development

No branches or pull requests
0 participants