On Tue, Apr 10, 2012 at 2:12 AM,  <msh...@math.vt.edu> wrote:
> Nicolas,
>
> The queue applies now :)
>
>> I am about to push another reviewer's patch which fixes some little
>> documentation compilation errors.
>
> I'll fold it in. Thanks!
>
>> By the way: ``apply_demazure_simple_reflection`` does the same as
>> ``apply_simple_projection``, right?
>
> Yes. I never looked at this method because it sounds different
> than what I was doing. I want to change the option
> "toward_max" to "length_increasing". Will anyone be bothered by this?

"toward_max" is used in

/categories/coxeter_groups.py
/monoids/bi_hecke_monoid.py
/combinat/j_trivial_monoids.py

I would also vote for "length_increasing" but we would have to change
it in the various places.

> If not I will delete apply_demazure_simple_reflection
> and use apply_simple_projection instead, with the renamed
> option length_increasing.

+1

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.

Reply via email to