#8119: Rename change the hash value of some objects
------------------------------+---------------------------------------------
Reporter: hivert | Owner: tbd
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.7
Component: misc | Keywords:
Author: | Upstream: N/A
Reviewer: Florent Hivert | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Changes (by hivert):
* status: needs_review => needs_work
* reviewer: => Florent Hivert
Comment:
> hivert: do you want to review this ticket?
Sure ! I completely forgot about this one. Sorry !
They are a few place where we should remove the bad implementation using
{{{__repr__}}} since they all inherits from {{{CategoryObject}}}:
{{{
popcorn-*ge-combinat/sage $ grep "hash(self.__repr__())" **/*.py*
groups/group.pyx: return hash(self.__repr__())
modules/module.pyx: return hash(self.__repr__())
modules/module.pyx: return hash(self.__repr__())
rings/polynomial/multi_polynomial_libsingular.pyx: return
hash(self.__repr__())
rings/ring.pyx: return hash(self.__repr__())
structure/sage_object.pyx: return hash(self.__repr__())
}}}
I don't have time to do it right now. I'll do it soon if you don't beat
me.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8119#comment:7>
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.
