#10130: Revamp __hash__, __cmp__ and __richcmp__
---------------------------+------------------------------------------------
Reporter: roed | Owner: robertwb
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.6.1
Component: coercion | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
---------------------------+------------------------------------------------
There are a number of confusing and non-optimal features of the way Sage
currently handles hashing and comparison.
* Because of Python, if you're writing a Cython class and you override
one of these functions, you must redefine the others as well. This is
easy to forget and confuses new users.
* The comparison infrastructure in sage.structure.element predates cpdef,
and could be made far less confusing with cpdef.
* hashes of parents are used extensively in the coercion framework, so
speed is quite important. But since parents are usually written in
Python, the current model will always have at least a dictionary lookup
(for example, in finding a cached `self.__hash`). And often parents don't
override the default hashing code and they fall back to slow `__repr__`
methods.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10130>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
