#9856: improve `augment` method for sparse matrices
----------------------------------+-----------------------------------------
   Reporter:  ylchapuy            |       Owner:  jason, was  
       Type:  enhancement         |      Status:  needs_review
   Priority:  major               |   Milestone:  sage-4.6    
  Component:  linear algebra      |    Keywords:              
     Author:  Yann Laigle-Chapuy  |    Upstream:  N/A         
   Reviewer:                      |      Merged:              
Work_issues:                      |  
----------------------------------+-----------------------------------------
Changes (by ylchapuy):

  * status:  needs_info => needs_review


Comment:

 It's a speed improvement. Here's the example:

 {{{
 sage: m = identity_matrix(QQ, 1000, sparse=True)
 sage: timeit('m.augment(m)')
 }}}

 BEFORE
 {{{
 5 loops, best of 3: 368 ms per loop
 }}}
 AFTER
 {{{
 625 loops, best of 3: 1.2 ms per loop
 }}}

 And we are not loosing anything for small cases:
 {{{
 sage: m = identity_matrix(QQ, 5, sparse=True)
 sage: timeit('m.augment(m)')
 }}}

 BEFORE
 {{{
 625 loops, best of 3: 198 µs per loop
 }}}
 AFTER
 {{{
 625 loops, best of 3: 197 µs per loop
 }}}

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