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:  sage6.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: 
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
Component:  basic arithmetic → algebra 

Dependencies:  #19330 → #19362 
comment:2 Changed 7 years ago by
Branch:  → u/jdemeyer/qqbar_polynomial_root____allow_approximate_root 

comment:3 Changed 7 years ago by
Commit:  → 41aaf04d4d6dd021a53038c9410f568cc8977958 

Status:  new → needs_review 
comment:4 Changed 7 years ago by
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
Branch:  u/jdemeyer/qqbar_polynomial_root____allow_approximate_root → u/cremona/19356 

Commit:  41aaf04d4d6dd021a53038c9410f568cc8977958 → afa179869d23b9aa4c3c42b807858d766030c997 
New commits:
afa1798  Merge branch 'develop (7.1.beta3)' into 19356

comment:6 Changed 7 years ago by
I merged with current develop (7.1.beta3). One minor doctest change was needed in QQbar. I hope I did not break etiquette ('ticketetiquette'!) by doing so and pushing the branch with my name on...
comment:8 Changed 4 years ago by
Keywords:  qqbar added 

Note: See
TracTickets for help on using
tickets.
New commits:
Move refine_root() to refine_root.pyx
Improve refine_root()
Allow approximate root in polynomial_root()