#12887: Optional args in Matrix_Mod submatrix method
---------------------------------------------------+------------------------
       Reporter:  dscharles                        |         Owner:  jason, was 
    
           Type:  defect                           |        Status:  
positive_review
       Priority:  minor                            |     Milestone:  sage-5.1   
    
      Component:  linear algebra                   |    Resolution:             
    
       Keywords:  matrix, finite field, submatrix  |   Work issues:             
    
Report Upstream:  N/A                              |     Reviewers:  Martin 
Albrecht
        Authors:  David Sanchez Charles            |     Merged in:             
    
   Dependencies:                                   |      Stopgaps:             
    
---------------------------------------------------+------------------------

Comment (by dscharles):

 I'm sorry I didn't realized before, but it also was necessary to change
 the output in order to work properly.
 (Should I open another ticket?)

 Before the last patch:
 {{{
 #!python
 sage: A = matrix([[1,1],[2,2],[3,3]])
 sage: B = matrix(Zmod(2), [[1,1],[2,2],[3,3]])
 sage: A.submatrix(2,2)
 []
 sage: A.submatrix(2,2).ncols()
 0
 sage: A.submatrix(2,2).nrows()
 1
 sage: B
 []
 sage: B.submatrix(2,2).ncols()
 0
 sage: B.submatrix(2,2).nrows()
 0
 }}}

 After the patch
 {{{
 #!python
 sage: B.submatrix(2,2)
 []
 sage: B.submatrix(2,2).ncols()
 0
 sage: B.submatrix(2,2).nrows()
 1
 }}}

 With that done, the smith_form method worked with some (easy) examples.

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