Hi Christian, > do you consider the following a bug? > > |sage: A = Core([2],4) > sage: B = Core([2],5) > sage: hash(A) == hash(B) > True| > > The hash of a core only depends on the list and not on the core. Since the > same list considered as an x- and as a y-core are fundamentally different > objects, I would rather suggest to have the hash > depending not only on the list. > > If others agree, I will provide a patch fixing that...
Yes, by all means, please fix that! > Thanks, Andrew! > > > sage: Core([2],4) is Partition([2]).core(4) > False > > > I agree with all what you write, except that I would not expect this to be > true since two instances of the same core are not the same object (thus do > not lie in the same place in space, thus "X is Y" > is not True). I agree with Christian since the two objects have different parents sage: (Partition([2]).to_core(4)).parent() 5-Cores of length 2 sage: Core([2],4).parent() 4-Cores of length 2 If you make other changes, it would be good to stay consistent with http://front.math.ucdavis.edu/1301.3569 or if it is absolutely necessary to change something that breaks the code in the book, please let us know, so we can fix it there as well. Best, Anne -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-combinat-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-combinat-devel@googlegroups.com. Visit this group at http://groups.google.com/group/sage-combinat-devel. For more options, visit https://groups.google.com/groups/opt_out.
