#10803: critical bug in real_roots
-------------------------+--------------------------------------------------
   Reporter:  zimmerma   |          Owner:  jason, jkantor
       Type:  defect     |         Status:  new           
   Priority:  critical   |      Milestone:  sage-4.7.3    
  Component:  numerical  |       Keywords:  real roots    
Work_issues:             |       Upstream:  N/A           
   Reviewer:             |         Author:                
     Merged:             |   Dependencies:                
-------------------------+--------------------------------------------------

Comment(by jdemeyer):

 Replying to [comment:12 zimmerma]:
 > I'm concerned by the fact that this ticket was delayed twice (first from
 4.7 to 4.7.1, now
 > from 4.7.2 to 4.7.3, btw why is there no trace of the change from 4.7.1
 to 4.7.2) and moreover
 > the priority is now changed from blocker to critical, without any
 justification.
 Well, it seems nobody really cares much to solve this problem.  I have
 glanced at the code and it's also hard to understand what is going on.

 Personally I don't think this should a "blocker".  Simple mathematical
 bugs are almost never blockers.  This `real_roots()` code is also not the
 default real-root computing code.  If you simply do `pol.roots(ring=RR)`,
 this will use PARI and not this buggy code (there could still be bugs in
 PARI though).

 > Is Carl Witty (who wrote the real_roots code as far as I know) still out
 there?
 AFAIK, he is not really into Sage development anymore.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10803#comment:13>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to