#14054: Cythoned UniqueRepresentation
-----------------------------------------------+----------------------------
Reporter: SimonKing | Owner: tbd
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.7
Component: performance | Resolution:
Keywords: cython UniqueRepresentation | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Simon King | Merged in:
Dependencies: #14017 | Stopgaps:
-----------------------------------------------+----------------------------
Comment (by SimonKing):
While we are at speeding up `UniqueRepresentation`, I think we should
actually refactor it. Namely, `UniqueRepresentation` serves two purposes,
namely (1) caching, and (2) comparison and hash by identity.
Some classes misuse `UniqueRepresentation` by only using feature (1),
overriding comparison in a way that violates the unique representation
condition. See my monologue at
[https://groups.google.com/forum/?fromgroups=#!topic/sage-devel/EOWWTK-
bUm8 sage-devel].
I suggest to split the two features.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:4>
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.
