#14054: Cythoned UniqueRepresentation
-----------------------------------------------+----------------------------
Reporter: SimonKing | Owner: tbd
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.8
Component: performance | Resolution:
Keywords: cython UniqueRepresentation | Work issues: Fix
non-uniqueness trouble of combinatorial free modules
Report Upstream: N/A | Reviewers: Travis
Scrimshaw
Authors: Simon King | Merged in:
Dependencies: #14017, #6495, #14182 | Stopgaps:
-----------------------------------------------+----------------------------
Changes (by SimonKing):
* status: needs_work => needs_review
* dependencies: #14017, #6495 => #14017, #6495, #14182
Comment:
I think I fixed all the remaining issues. This includes a fix to my
recently added coercion tutorial (the number of inherited methods in two
examples changes).
Concerning combinatorial free modules: They used to violate the unique
parent condition, even though they inherited from `UniqueRepresentation`.
I have demonstrated that this was a bug, when combining the failure to be
unique with a cache on the tensor product construction. It is
automatically fixed with the changes to `UniqueRepresentation` introduced
in this patch, and gave rise to a new doctest.
Needs review!
Apply trac14054_fast_methods-5.8.patch
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:58>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
