#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):
From [https://groups.google.com/forum/?fromgroups=#!topic/sage-devel
/EOWWTK-bUm8 sage-devel], I understand that people think that separating
the cache feature from the uniqueness feature of `UniqueRepresentation` is
a good idea.
However, in my old patch, I was ''enforcing'' uniqueness behaviour for
instances of `UniqueRepresentation`. This isn't pythonic. Hence, I do
differently in the new patch version.
'''__Question__'''
In the current implementation, inheritance from `UniqueRepresentation`
will overload rich comparison (==, >=, !=, etc.) inherited from a base
class, but it will not overload comparison (cmp). Do you think that
''both'' should be overloaded?
Patchbot reported two failures with the previous patch version. I guess
that's because of an additional dependency. So, as soon as I have a decent
internet connection, I'll download the latest beta, and rebase on top of
it.
Apply trac14054_fast_methods.patch
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:6>
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.
