#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:
Report Upstream: N/A | Reviewers: Travis
Scrimshaw
Authors: Simon King | Merged in:
Dependencies: #14017 | Stopgaps:
-----------------------------------------------+----------------------------
Comment (by nbruin):
The issue is that if someone wants to define
{{{
class C(WithEqualityById):
def __cmp__(self,other):
return -1
}}}
they will find that
{{{
sage: a=C()
sage: a < a
False
}}}
which might surprise them, because they thought that `WithEqualityById`
only affected (in)equality testing and let ordering comparisons fall
through to `__cmp__` if implemented. With the current code, this is not
the case if the two arguments are identical.
There have been extensive discussions about what inequality testing SHOULD
be in sage and for the most part the implementation here is staying clear
of the topic (which I think is a good thing). There's just this one small
optimization that's probably usually OK, but if left out makes for much
more predictable behaviour.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:19>
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.
