Hello Christian, I just pushed a change to composition.py which breaks your patch.
Since you are actively working on this, I decided not to rebase your patch but instead disabled it. It is a very easy rebase in the import statements. Also, I disabled Mike's patch, which depends on yours. Please re-enable it as well when you enable yours: pp_function_for_dyck_words-mz.patch Franco -- On Thu, Jul 5, 2012 at 4:40 AM, Christian Stump <christian.st...@gmail.com> wrote: > Hi Martin, > >> one patch per class? > > that's completely up to you, I guess > >> Another question: is it important to keep the number of statistics >> small? I.e., if a statistic can be constructed via a map, is it better >> *not* to implement it? (Eg., saliances and right-to-left-minima.) > > I usually do implement them, and add it then to the black list of the > combinatorial map. Otherwise we would even not have maj or inv since > the bijection sending maj to inv is also implemented. > >> How about aliases? (I would like to have right-to-left-maxima be an >> alternative name to saliances...) > > we already use them some times. only the decorated one will appear, > not the alias. > > best, christian > > -- > You received this message because you are subscribed to the Google Groups > "sage-combinat-devel" group. > To post to this group, send email to sage-combinat-devel@googlegroups.com. > To unsubscribe from this group, send email to > sage-combinat-devel+unsubscr...@googlegroups.com. > For more options, visit this group at > http://groups.google.com/group/sage-combinat-devel?hl=en. > -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to sage-combinat-devel@googlegroups.com. To unsubscribe from this group, send email to sage-combinat-devel+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sage-combinat-devel?hl=en.