Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-08-01 Thread Nicolas M. Thiery
On Mon, Jul 30, 2012 at 09:04:44PM -0700, Anne Schilling wrote: Since there were some nonlocal changes in sage-5.1, I think it would be best if all developers now switch to sage-5.2 and we focus our efforts on rebasing all patches to this new version of sage (instead of trying to keep

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-31 Thread Daniel Bump
I have rebased 9265 so that it applies on 5.1 but I haven't uploaded it as I wanted to rename it and make sure that I don't clobber anything else in the 5.1 or 5.2 queues. The main problem is that even though the queue applies on 5.2 sage does not run due -- so far -- to incorrect

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-31 Thread Andrew Mathas
Sorry, I have pushed the patch onto the queue and reinstated 9265 in the series file. Please let me know if there are any problems. Btw, on trac the patchbot is applying both the old patch and the new one. Could some please tell me how I get it to ignore the old patch. Cheers, Andrew On

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-31 Thread Franco Saliola
On Tue, Jul 31, 2012 at 9:01 PM, Andrew Mathas andrew.mat...@sydney.edu.au wrote: Sorry, I have pushed the patch onto the queue and reinstated 9265 in the series file. Please let me know if there are any problems. Btw, on trac the patchbot is applying both the old patch and the new one. Could

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-31 Thread Franco Saliola
On Tue, Jul 31, 2012 at 10:25 PM, Franco Saliola sali...@gmail.com wrote: On Tue, Jul 31, 2012 at 9:01 PM, Andrew Mathas andrew.mat...@sydney.edu.au wrote: Sorry, I have pushed the patch onto the queue and reinstated 9265 in the series file. Please let me know if there are any problems. Btw,

[sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Andrew Mathas
I have rebased Jason's patch so that it apples to 5.2-rc0. I also uploaded the new patch to trac, where I had to rename the patch as trac would not give me permission to delete some one else's patch. The patch probably won't apply cleanly to 5.2-rc0 for trivial reasons (white space) as it

[sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Andrew Mathas
ps. I put a +5.2 guard on the patch concrete_combinatorial_statistics_and_maps-cs.patch as it didn't apply cleanly to files that I hadn't touched. (I also guarded my patch trac_X-Kleshchev-partitions-am.patch which I forgot to do when I uploaded it earlier). Andrew -- You received this

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Daniel Bump
I also uploaded the new patch to trac, where I had to rename the patch as trac would not give me permission to delete some one else's patch. The patch probably won't apply cleanly to 5.2-rc0 for trivial reasons (white space) It should apply cleanly to 5.2. Assuming you fix that and repost

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Christian Stump
It should apply cleanly to 5.2. Assuming you fix that and repost it, unfortunately, 9265 does not apply anymore on 5.1... 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

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Daniel Bump
Christian Stump wrote: unfortunately, 9265 does not apply anymore on 5.1... I think a lot of patches are broke on 5.1 right now. Here is a partial list. trac_5457-review-as.patch trac_9265_tableaux_categories_jb.patch trac_12774-coxeter-ms.patch

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Anne Schilling
Sorry, I forgot the guard for trac_5457-review-as.patch and just added it! Anne On 7/30/12 8:12 PM, Daniel Bump wrote: Christian Stump wrote: unfortunately, 9265 does not apply anymore on 5.1... I think a lot of patches are broke on 5.1 right now. Here is a partial list.

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Anne Schilling
Hi All! Since there were some nonlocal changes in sage-5.1, I think it would be best if all developers now switch to sage-5.2 and we focus our efforts on rebasing all patches to this new version of sage (instead of trying to keep backwards compatibilty for much longer). Best, Anne On 7/30/12

Re: [sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-30 Thread Andrew Mathas
Dear Dan and Christian, I have rebased 9265 so that it applies on 5.1 but I haven't uploaded it as I wanted to rename it and make sure that I don't clobber anything else in the 5.1 or 5.2 queues. The main problem is that even though the queue applies on 5.2 sage does not run due -- so far --

[sage-combinat-devel] Re: Trac 9265: Remove `CombinatorialClass` from sage.combinat.tableau

2012-07-24 Thread Dima Pasechnik
I just marked it as needs work. it needs a (trivial, hopefully) rebase for Sage 5.2.rc0. Dima On Tuesday, 24 July 2012 10:19:56 UTC+8, Andrew Mathas wrote: Hi Jason, What is the status of your patch #9265? It is not marked as needing review and I'd actually assumed it was already merged