Re: [sage-combinat-devel] Newbie review question

2012-07-30 Thread Andrew Mathas
Vincent and Dan thanks for the replies. Just to clarify, Vincent said: Dependencies should be used in trac (it is one of the field in the definition of the ticket and consists an a (possibly empty) sequence of patches). but, as Dan implied, the dependency field in trac if for when a patch

[sage-combinat-devel] Re: Newbie review question

2012-07-30 Thread Simon King
Hi Andrew, On 2012-07-30, Vincent Delecroix 20100.delecr...@gmail.com wrote: Hi Andrew, Dependencies should be used in trac (it is one of the field in the definition of the ticket and consists an a (possibly empty) sequence of patches). An example at [1]. When a patch is ready in

Re: [sage-combinat-devel] Newbie review question

2012-07-30 Thread Daniel Bump
I have rebased Jason's patch and against 5.2-rc0.I am checking the doctests etc. When I am done I will repost the patch to trac and the queue. There was a message on sage-release that Sage 5.2 is now available. It would be good to test it on that.

Re: [sage-combinat-devel] Chromatic Symmetric Function ... Function

2012-07-30 Thread Nicolas M. Thiery
Hi Keeler, On Wed, Jul 25, 2012 at 05:14:41PM -0700, Keeler Russell wrote: I've been working on a function to calculate the chromatic symmetric function (a.k.a. CSF) of a graph as part of a related research project. You can find the code here: https://github.com/keeler/csf

[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 --