#20681: Cythonize the special methods in the categories that handle coercion in
arithmetic
-------------------------------------+-------------------------------------
Reporter: nthiery | Owner:
Type: enhancement | Status: needs_info
Priority: major | Milestone: sage-7.3
Component: categories | Resolution:
Keywords: performance | Merged in:
Authors: Nicolas M. Thiéry | Reviewers: Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/categories/cythonize_arithmetic_special_methods-20681|
de62437f434f282dd41256ff9529d4aca3d679fd
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jdemeyer):
* status: positive_review => needs_info
Comment:
I understand the point of this patch, but I think it's not the right thing
to do. The methods you define now in `coercion_methods.pyx` are just
copies of methods available in various subclasses of `Element` anyway
(e.g. `ModuleElement`, `RingElement`).
I think the right thing to do is just to get rid of the category framework
for these arithmetic methods and just define them in `Element`. That will
be even faster than the current patch.
--
Ticket URL: <http://trac.sagemath.org/ticket/20681#comment:15>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
