#6237: repeated roots with roots(CDF, multiplicities=False)
--------------------------------+-------------------------------------------
Reporter: ncalexan | Owner: somebody
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.3.1
Component: basic arithmetic | Keywords: roots CDF multiplicities
Work_issues: | Author: Alex Ghitza
Upstream: N/A | Reviewer: Karl-Dieter Crisman
Merged: |
--------------------------------+-------------------------------------------
Changes (by kcrisman):
* status: needs_review => positive_review
* reviewer: => Karl-Dieter Crisman
Comment:
Looks good. Passes relevant tests.
I did find something else weird, perhaps just my install is somehow
corrupt... Did you try the original thing from the description as well?
It seems to be broken at the calculation of the discriminant both before
and after the patch. I get an error message about PariError(8). However,
this does not appear on sage.math, so I assume something weird happened in
my local install. But I thought I'd mention it in case you find it.
Anyway, positive review!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6237#comment:5>
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.
