#14519: Cythonize ElementWrapper and make parent the first argument
-----------------------------------------+----------------------------------
Reporter: tscrim | Owner: tscrim
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.11
Component: performance | Resolution:
Keywords: cython ElementWrapper | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Travis Scrimshaw | Merged in:
Dependencies: #14143 #14516 | Stopgaps:
-----------------------------------------+----------------------------------
Changes (by tscrim):
* status: needs_work => needs_review
* work_issues: fix segfault in UCF =>
Comment:
The UCF segfault was due to it inheriting from both `FieldElement` and
`ElementWrapper`. In particular, the `_add_()` / `_mul_()` / etc.
functions of the UCF elements were not being called when `ElementWrapper`
is an extension class. It also segfaults when I have it inherit from
`CombinatorialFreeModuleElement`, which is what it wraps. I'll see if I
can reduce it down to a simple test case and post it on another ticket and
sage-devel since I don't know if this is a python, cython, or sage bug...
Also #11931 can be closed as a duplicate.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14519#comment:3>
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/groups/opt_out.
