Yes they should definitely be consistent -- sorry, I misread your email and thought that you wanted the method for skew tableau to be the reverse of that for tableaux....not enough coffee (or maybe too much:).
As it currently stands there are many independent tableaux classes. Ideally, methods like this should live in a common base class so consistency is enforced and we don't re-implement them in each class. Andrew On Tuesday, 29 November 2016 09:33:52 UTC+11, Anne Schilling wrote: > > Hi Andrew, > > If someone had just copied the code from Tableaux to SkewTableaux, > then at least it would be consistent. I think it is really bad, > if a method changes convention with a parent, given that one > could be a subclass of the other. > > Best, > > Anne > > On 11/28/16 1:23 PM, Andrew wrote: > > Hi Anne, > > > > One of the problems with tableaux is that almost every possible > variation of any definition is used by some one. With this in mind we > probably should implement variations whenever possible. I don't > > know anything about the history of this particular method (git blame > might help), but I suspect that some one just copied the code from tableaux > to skew tableaux. I certainly have no objection either > > way as I don't use this method. > > > > Andrew > > > > On Tuesday, 29 November 2016 05:41:52 UTC+11, Anne Schilling wrote: > > > > Hi Combinat developers, > > > > Who came up with these conventions: > > > > sage: s=Tableau([[4,5],[6,7],[7],[8]]) > > sage: s.to_word_by_column() > > word: 876475 > > sage: s=SkewTableau([[4,5],[6,7],[7],[8]]) > > sage: s.to_word_by_column() > > word: 574678 > > > > The column word of a skew tableau is the reverse of that > > of a regular tableau. I think we should make the convention > > the same (using the tableau convention!!!). > > > > What do you think? > > > > 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 https://groups.google.com/group/sage-combinat-devel. For more options, visit https://groups.google.com/d/optout.
