#4260: use LinBox as native matrix representation for dense matrices over GF(p)
----------------------------------+-----------------------------------------
   Reporter:  malb                |          Owner:  cpernet                    
               
       Type:  enhancement         |         Status:  needs_review               
               
   Priority:  major               |      Milestone:  sage-4.7.2                 
               
  Component:  linear algebra      |       Keywords:  linbox, linear algebra, 
sd32, sd34        
Work_issues:  improve echelonize  |       Upstream:  N/A                        
               
   Reviewer:  Burcin Erocal       |         Author:  Burcin Erocal, Martin 
Albrecht, Rob Beezer
     Merged:                      |   Dependencies:                             
               
----------------------------------+-----------------------------------------
Changes (by burcin):

  * keywords:  linbox, linear algebra, sd32 => linbox, linear algebra,
               sd32, sd34
  * reviewer:  => Burcin Erocal


Comment:

 attachment:trac_4260-minor_fixes.patch
  * makes some cosmetic changes and
  * fixes a possible memory leak if allocation of these matrices fail.

 I read through the patches and the resulting code. All looks good to me.
 Please switch this to positive review if my patch is ok.

 Thanks everyone for finally finishing this off.

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