#9720: Add random echelonizable matrices to matrix/constructor.py
------------------------------+---------------------------------------------
   Reporter:  bwonderly       |       Owner:  jason, was
       Type:  enhancement     |      Status:  needs_info
   Priority:  major           |   Milestone:  sage-4.5.3
  Component:  linear algebra  |    Keywords:            
     Author:  Billy Wonderly  |    Upstream:  N/A       
   Reviewer:  David Joyner    |      Merged:            
Work_issues:                  |  
------------------------------+---------------------------------------------

Comment(by mhansen):

 Replying to [comment:15 rbeezer]:
 > Would you pass some sort of selector in {{{*args}}}, then make these
 methods of some class of matrices?  How would the documentation be
 immediately or easily accessible?  Is there a model we can follow
 someplace else in Sage?

 You can leave the functions as they are -- just don't import them into the
 global namespace.  Then, if you do,

 {{{
 sage: a = random_matrix(QQ, 3, 3, rref=True)
 sage: b = random_matrix(QQ, 3, 3, echelonizable=True)
 }}}

 which just delegate to the appropriate function.  You can add a reference
 to those functions in the docstring of {{{random_matrix}}}.

 Also, the documentation is not clear on why there is special casing for ZZ
 and QQ, and that these methods don't work for inexact rings.  Although,
 you could argue that if you have one of these matrices over {{{ZZ}}} you
 could just convert all the entries to {{{RR}}} and things "should" be
 fine.

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