#804: Matrix should not inherit from AlgebraElement
-------------------------------------+-------------------------------------
       Reporter:  roed               |        Owner:  was
           Type:  defect             |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.2
      Component:  linear algebra     |   Resolution:
       Keywords:  AlgebraElement,    |    Merged in:
  Matrix                             |    Reviewers:
        Authors:  Peter Bruin        |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  0fd3d318a230b2865cf331c2c696c9d05ceba487
  u/pbruin/804-Matrix_inheritance    |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Peter Bruin', 'oldvalue': ''}):

 * status:  new => needs_review
 * commit:   => 0fd3d318a230b2865cf331c2c696c9d05ceba487
 * branch:   => u/pbruin/804-Matrix_inheritance
 * author:   => Peter Bruin


Old description:

> People just don't want to fix it because you'll have to rebuild
> everything after editing element.pxd.

New description:

 This ticket makes `Matrix` only inherit from `ModuleElement`.

 Warning: it will cause practically all Cython files to be rebuilt.

 See also #15215 (duplicate of this ticket).

--

Comment:

 Given that this ticket has been open for more than seven years, it turned
 out to be surprisingly straightforward.  There is one small
 simplification: `Matrix` used to have two identical methods `_mul_()` and
 `__mul__()`, now it only needs `__mul__()`.  On the other hand, new (but
 very straightforward) `__pow__()` and `__div__()` methods were required.

--
Ticket URL: <http://trac.sagemath.org/ticket/804#comment:11>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to