#11027: Schur matrix decomposition over RDF/CDF
----------------------------+-----------------------------------------------
Reporter: rbeezer | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-4.7
Component: linear algebra | Resolution:
Keywords: | Author: Rob Beezer
Upstream: N/A | Reviewer: Martin Raum
Merged: | Work_issues:
----------------------------+-----------------------------------------------
Changes (by rbeezer):
* status: needs_work => needs_review
Old description:
> Wraps the `schur()` function from `SciPy`.
>
> Apply trac_11027-schur-decomposition-v2.patch
New description:
Wraps the `schur()` function from `SciPy`.
'''Apply:'''
1. [attachment:trac_11027-schur-decomposition-v2.patch]
1. [attachment:trac_11027-schur-decomposition-doctest.patch]
--
Comment:
Well, it looks like this was just carelessness on my part, and not any
kind of platform-variability problem. On other doctests, which seem to
pass for the various testers, I have code like
{{{
sage: T = T.zero_at(1.0e-12).change_ring(RDF)
sage: T.round(6)
}}}
The first line was missing on this one problem test - the doctest patch
just adds in such a line. Likely a cut/paste when there should have been
a copy/paste.
With the new patch, this passes tests on my 64-bit Ubuntu machine and on
sage.math. I'm in the process of testing on `SkyNet/cleo`. I'm confident
this fix should be OK, but "once burned, twice shy."
This is ready for review, but I'd like to have the cleo result before it
gets flipped to positive review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11027#comment:20>
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.
