#19016: A more naive sage.structure.element.__hash__
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_review
Type: | Milestone: sage-6.9
defect | Resolution:
Priority: | Merged in:
blocker | Reviewers:
Component: misc | Work issues:
Keywords: | Commit:
Authors: | 754dc5794a1a7004c8844cf7cfb64220957c36a5
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
u/ncohen/19016 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by nbruin):
Replying to [comment:10 ncohen]:
> What is your position for "constant hash" Vs "we change nothing and the
bug stays"?
"Change nothing". I think it's unfortunate that hashing by repr will
usually work quite well in cases where a hash is possible: a hash usually
requires a canonical form to compute the hash from and that canonical form
will usually be used for printing, because that's nice. So in practice,
the default hash (although much too slow to be a reasonable hash) will
probably work when it should.
In cases where the current default hash *doesn't* work, I think that
having it as it is now will lead to better detection than when we change
it to a default hash. When people find this in particular cases, the
solution is clear: actively disable the hash.
Having to *disable* something is of course the wrong way around: the
default hash shouldn't be there in the first place. But putting a constant
0 hash there instead doesn't make transitioning to the desired situation
any easier.
--
Ticket URL: <http://trac.sagemath.org/ticket/19016#comment:11>
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.
For more options, visit https://groups.google.com/d/optout.
