#18246: remove naive __hash__ from SageObject
-------------------------+-------------------------------------------------
Reporter: | Owner:
vdelecroix | Status: new
Type: | Milestone: sage-6.9
defect | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Volker Braun
coercion | Work issues:
Keywords: | Commit:
Authors: | 4d8ec2a1282b24064f93bc8bea5fe23ec08f1750
Vincent Delecroix | Stopgaps:
Report Upstream: N/A |
Branch: |
public/18246 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by tscrim):
I actually don't like these changes:
{{{#!diff
-class VectorPartitions(Parent, UniqueRepresentation):
+class VectorPartitions(UniqueRepresentation, Parent):
}}}
Why does `UniqueRepresentation` have to come before `Parent`? If this how
things must be, then we should create a class `URParent` (or a better
name), use that all over Sage to avoid this issue. (FYI: Before this
ticket, there are 107 classes which use `UR, Parent` and 107 which use
`Parent, UR`.)
Also can someone explain to me why are `RiggedConfigurations` now showing
machine dependent iteration order?
--
Ticket URL: <http://trac.sagemath.org/ticket/18246#comment:39>
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.
