#9760: Possible numerical noise doctest failure in sage/matrix/matrix2.pyx on t2
-----------------------+----------------------------------------------------
Reporter: mpatel | Owner: drkirkby
Type: defect | Status: new
Priority: blocker | Milestone: sage-4.5.3
Component: solaris | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------+----------------------------------------------------
Comment(by jhpalmieri):
Replying to [comment:5 drkirkby]:
> Can you try backing up the additional shared libraries, then see if the
tests pass or fail in the same way. It could well totally mess up the
build, as if a shared library is missing, it may fail to work. But if it
still works, but fails, that means the change in the static library has
caused this.
I'm not sure what you mean by this. For the tests with the old ATLAS build
and the new one, I changed all of the libraries built by ATLAS:
libatlas.a, libatlas.so, ... (7 altogether, 4 static and 3 shared, since
liblapack.so is not built). Do you want me to just delete the .so files
instead? Sage doesn't start in that case.
> If I have understood you correctly, to get this to pass, a change from
{{{
abs((imag(e)) < 1e-15
}}}
> to
{{{
abs((imag(e)) < 1.1e-15
}}}
Right, except it's actually changing from {{{imag(e) < 1e-15}}} (with no
"abs") to {{{abs(imag(e)) < 1.1e-15}}}. The test does pass with this
change. It would be safest to not add the "abs" right now: with that
change, it would clearly still pass wherever it worked before. Adding the
"abs" changes what the test is actually testing, though, and would require
evaluation on lots of platforms: for all we know, there is numerical noise
somewhere else which gives a relatively large negative imaginary part for
one of the eigenvalues. Should we make this change on this ticket or
elsewhere?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9760#comment:6>
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.
