Re: [sage-combinat-devel] combinatorial statistics

2012-07-05 Thread Franco Saliola
On Thu, Jul 5, 2012 at 12:49 PM, Christian Stump wrote: >> I just pushed a change to composition.py which breaks your patch. > > I rebased the patch and also enabled Mike's patch again. But now, I > get the following error message after applying the queue, so I don't > push the changed... > > Any

Re: [sage-combinat-devel] combinatorial statistics

2012-07-05 Thread Christian Stump
> I just pushed a change to composition.py which breaks your patch. I rebased the patch and also enabled Mike's patch again. But now, I get the following error message after applying the queue, so I don't push the changed... Any idea (I am using 5.0)?

Re: [sage-combinat-devel] combinatorial statistics

2012-07-05 Thread Franco Saliola
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: [sage-combinat-devel] combinatorial statistics

2012-07-05 Thread Christian Stump
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