#10604: Rewrite diagonal matrix constructor
------------------------------+---------------------------------------------
   Reporter:  rbeezer         |       Owner:  ddrake    
       Type:  defect          |      Status:  needs_work
   Priority:  minor           |   Milestone:  sage-4.6.2
  Component:  linear algebra  |    Keywords:            
     Author:  Rob Beezer      |    Upstream:  N/A       
   Reviewer:  Dan Drake       |      Merged:            
Work_issues:                  |  
------------------------------+---------------------------------------------

Comment(by rbeezer):

 Replying to [comment:6 ddrake]:
 > I'm also seeing a doctest failure:
 >
 > {{{
 > [email protected]:/scratch/drake/days27/sage-4.6.1.rc1$ ./sage -t  devel
 /sage-main/sage/rings/number_field/number_field_ideal.py

 Smith Form seems to be unhappy about getting a sparse matrix:

 {{{
 sage: B=diagonal_matrix(ZZ, [3,4])
 sage: B.is_sparse()
 True
 sage: B.smith_form()
 ---------------------------------------------------------------------------
 TypeError                                 Traceback (most recent call
 last)

 <snip>

 TypeError: unable to coerce <class 'sage.rings.ideal.Ideal_pid'> to an
 integer

 sage: C=B.dense_matrix()
 sage: C.smith_form()
 (
 [ 1  0]  [ 1  1]  [-1 -4]
 [ 0 12], [-4 -3], [ 1  3]
 )
 }}}

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