Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-12 Thread Nicolas M. Thiery
On Sat, Mar 10, 2012 at 02:04:44PM +0100, Christian Stump wrote: Christian: I had to slightly rebase trac_11187-finite_reflection_groups-cs.patch for the doctests I just added to apply_simple_reflection and friends. This time I rebased nonnesting_partitions-cs.patch which was touching

Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-10 Thread Nicolas M. Thiery
Hi Anne, On Fri, Mar 09, 2012 at 04:09:13PM -0800, Anne Schilling wrote: Impressive patch! Thanks for working on this. :-) Thanks so much for your very useful and quick review. I really appreciate that! Followup on http://trac.sagemath.org/sage_trac/ticket/6588 Cheers,

Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-10 Thread Nicolas M. Thiery
Christian: I had to slightly rebase trac_11187-finite_reflection_groups-cs.patch for the doctests I just added to apply_simple_reflection and friends. Cheers, Nicolas -- Nicolas M. ThiƩry Isil nthi...@users.sf.net http://Nicolas.Thiery.name/ -- You received this

Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-10 Thread Christian Stump
Hi -- as you are currently discussing root systems (including 6588, thanks a lot for working on that!): here are some issues that made it very hard for me to work with root systems for finite reflection groups: # the first and the second seem to behave very differently: sage: x =

Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-10 Thread Anne Schilling
Hi Christian, as you are currently discussing root systems (including 6588, thanks a lot for working on that!): here are some issues that made it very hard for me to work with root systems for finite reflection groups: # the first and the second seem to behave very differently: sage: x =

Re: [sage-combinat-devel] Re: the unfortunate case of affine type A twisted

2012-03-09 Thread Anne Schilling
Hi Nicolas, Impressive patch! Thanks for working on this. Here are some first comments: - In /sage/categories/affine_weyl_groups.py there is a new import from sage.categories.infinite_enumerated_sets import InfiniteEnumeratedSets which is not used. Please either remove this line or add