#10733: Faster echelon form code for matrix_modn_sparse
--------------------------------+-------------------------------------------
   Reporter:  tornaria          |       Owner:  was         
       Type:  enhancement       |      Status:  needs_review
   Priority:  major             |   Milestone:              
  Component:  linear algebra    |    Keywords:              
     Author:  Gonzalo Tornaria  |    Upstream:  N/A         
   Reviewer:                    |      Merged:              
Work_issues:                    |  
--------------------------------+-------------------------------------------
Changes (by tornaria):

  * status:  needs_work => needs_review


Comment:

 The patch doesn't add any new functionality.

 The function didn't have a doctest before.

 I've attached a patch (to be applied in top of the other two) which adds a
 test for the sparse echelon form. The test takes about 20 seconds, so it's
 labeled as long.

 What the test does is to construct a 1000x1000 sparse matrix with
 density=1%, and compute the echelon form using different values for
 "switch_density", and also using the dense matrix echelon form method. Of
 course everything should give the same echelon form. I'm not sure if it
 makes sense to test with smaller matrices.

 In any case, feel free to add more tests or modify the current one in any
 way you see fits.

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