#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.