#5510: [with patch, needs work] update M4RI interface
----------------------------+-----------------------------------------------
Reporter: malb | Owner: malb
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.0.1
Component: linear algebra | Keywords: m4ri, linear algebra
----------------------------+-----------------------------------------------
Comment(by malb):
Here is the code that used to be called:
{{{
def rank(self):
x = self.fetch('rank')
if not x is None: return x
if self._nrows == 0 or self._ncols == 0:
return 0
}}}
And here is the code that is called now:
{{{
def rank(self):
x = self.fetch('rank')
if not x is None:
return x
if self._nrows == 0 or self._ncols == 0:
return 0
}}}
i.e. they are identical and thus I have no idea to address this
performance regression. Maybe the fetch is more expensive from a subclass
or something? In any case, I don't think this should hold back the merge
of this ticket.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5510#comment:3>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
