#13045: Deprecate cholesky_decomposition() in favor of cholesky()
--------------------------------------------------+-------------------------
Reporter: rbeezer | Owner: jason, was
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-5.1
Component: linear algebra | Resolution:
Keywords: sd40.5 | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Rob Beezer | Merged in:
Dependencies: #12966, #13018, #11274, #13035 | Stopgaps:
--------------------------------------------------+-------------------------
Comment (by ddrake):
More seriously, the caching means that this broken, crappy function can
infect the new, correct function:
{{{
sage: r = matrix(CDF, 2, 2, [ 1, -2*I, 2*I, 0 ]); r
[ 1.0 -2.0*I]
[ 2.0*I 0.0]
sage: r[0,0] = 0
sage: r.cholesky_decomposition()
[ 0.0 0.0]
[NaN + NaN*I NaN + NaN*I]
sage: r.cholesky()
[ 0.0 0.0]
[NaN + NaN*I NaN + NaN*I]
}}}
The documentation makes it clear that `cholesky_decomposition` can return
bad answers, but the new function shouldn't be infected by that.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13045#comment:4>
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.
