#11608: RDF/CDF eigenvalues: symmetric matrices, multiplicities
----------------------------------+-----------------------------------------
       Reporter:  rbeezer         |         Owner:  jason, was  
           Type:  enhancement     |        Status:  needs_work  
       Priority:  minor           |     Milestone:  sage-5.0    
      Component:  linear algebra  |    Resolution:              
       Keywords:                  |   Work issues:              
Report Upstream:  N/A             |     Reviewers:  Volker Braun
        Authors:  Rob Beezer      |     Merged in:              
   Dependencies:                  |      Stopgaps:              
----------------------------------+-----------------------------------------
Changes (by vbraun):

  * status:  needs_review => needs_work
  * reviewer:  => Volker Braun


Comment:

 On Fedora 16 x86_64, I get
 {{{
 sage -t  devel/sage-main/sage/matrix/matrix_double_dense.pyx
 **********************************************************************
 File "/home/vbraun/opt/sage-5.0.rc0/devel/sage-
 main/sage/matrix/matrix_double_dense.pyx", line 1570:
     sage: A.eigenvalues(algorithm='symmetric', tol=1.0e-20)
 Expected:
     [(-2.0, 1), (-2.0, 2), (-2.0, 1), (1.0, 1), (1.0, 1), (1.0, 1), (1.0,
 1), (1.0, 1), (3.0, 1)]
 Got:
     [(-2.0, 1), (-2.0, 2), (-2.0, 1), (1.0, 1), (1.0, 1), (1.0, 2), (1.0,
 1), (3.0, 1)]
 }}}
 ATLAS is from Fedora, CPU is a Sandy-Bridge i7 quad-core.

 Minor nitpicks while we are at it:
   * Can you return a tuple (=immutable) instead of a list?
   * the symmetric docstring, how about: "converts `self` to a real matrix
 and applies the algorithm for Hermitian matrices". Note: technically you
 convert (explicit), not coerce (implicit).
   * in the hermitian docstring, how about: "uses the
 :meth:`~scipy.linalg.eigh` method from !SciPy, which applies only to real
 symmetric or complex Hermitian matrices. Since...". This would make it
 clear that the `syevr` implementation is used if possible.
   * the ".. warning::" block is not indented, so it is not typeset
 correctly.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11608#comment:8>
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