#19016: Better hash for Element
-------------------------------------+-------------------------------------
Reporter: ncohen | Owner:
Type: defect | Status: needs_work
Priority: blocker | Milestone: sage-6.9
Component: misc | Resolution:
Keywords: | Merged in:
Authors: Nils Bruin, | Reviewers:
Vincent Delecroix | Work issues:
Report Upstream: N/A | Commit:
Branch: public/19016 | a0f830bf8dc0cbdeb75e45ee7bfee7b4fad33768
Dependencies: #18246 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nbruin):
Replying to [comment:72 ncohen]:
> That is what stopgaps are made for: hard to fix quickly, and warn
against mistakes.
>
> I don't mind seing the status of this ticket change if we add a stopgap
in the next release. That will not change the behaviour or anything, nor
the doctests.
I'd be OK with that. When one ends up in
sage.structure.element.Element.__hash__ , performance will be down the
drain anyway, and the warning only prints once.
Incidentally, the hash of symbolic expressions is *not* this one.
Furthermore, the renaming feature mentioned in the ticket is highly
incompatible with sage's model of immutability and names having
mathematical significance. So resolving the title of the ticket will not
address several of the perceived issues mentioned in the description.
--
Ticket URL: <http://trac.sagemath.org/ticket/19016#comment:75>
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.
