Opened 7 years ago

Last modified 4 years ago

#19356 needs_work enhancement

QQbar.polynomial_root(): allow approximate root

Reported by: Jeroen Demeyer Owned by:
Priority: major Milestone: sage-6.9
Component: algebra Keywords: qqbar
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers:
Report Upstream: N/A Work issues:
Branch: u/cremona/19356 (Commits, GitHub, GitLab) Commit: afa179869d23b9aa4c3c42b807858d766030c997
Dependencies: #19362 Stopgaps:

Status badges

Description

Instead of only allowing an interval containing a root, an approximate root should also be accepted.

Change History (8)

comment:1 Changed 7 years ago by Jeroen Demeyer

Component: basic arithmeticalgebra
Dependencies: #19330#19362

comment:2 Changed 7 years ago by Jeroen Demeyer

Branch: u/jdemeyer/qqbar_polynomial_root____allow_approximate_root

comment:3 Changed 7 years ago by Jeroen Demeyer

Commit: 41aaf04d4d6dd021a53038c9410f568cc8977958
Status: newneeds_review

New commits:

2049f5aMove refine_root() to refine_root.pyx
3b34e49Improve refine_root()
41aaf04Allow approximate root in polynomial_root()

comment:4 Changed 7 years ago by John Cremona

I'm not sure why this never got reviewed, so I am reviewing it after rebasing it on 7.1.beta3 (actually merging not rebasing). The code looks good, but I suppose that #19362 needs finishing first. I am not sure what the needs_work issue is with that one?

comment:5 Changed 7 years ago by John Cremona

Branch: u/jdemeyer/qqbar_polynomial_root____allow_approximate_rootu/cremona/19356
Commit: 41aaf04d4d6dd021a53038c9410f568cc8977958afa179869d23b9aa4c3c42b807858d766030c997

New commits:

afa1798Merge branch 'develop (7.1.beta3)' into 19356

comment:6 Changed 7 years ago by John Cremona

I merged with current develop (7.1.beta3). One minor doctest change was needed in QQbar. I hope I did not break etiquette ('ticket-etiquette'!) by doing so and pushing the branch with my name on...

comment:7 Changed 6 years ago by Clemens Heuberger

Status: needs_reviewneeds_work

merge conflict

comment:8 Changed 4 years ago by Jean-Philippe Labbé

Keywords: qqbar added
Note: See TracTickets for help on using tickets.