I posted a positive review on the trac server.

Anne

bump wrote:
I made these changes and also added Nicolas as an author. The
reposted patch is on the trac server.

Dan

If you (Dan and Nicolas) would like to submit the patch jointly,
I am happy to review. Here are a couple of small comments:

- Perhaps it would be better to name the file
   sage.algebras.iwahori_hecke_algebra  or
   sage.algebras.iwahori_hecke ?

- line 221 in algebra/iwahori.py
   coxeter group -> Coxeter group

Other than that, I would be happy to set a positive review.
-- 
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-de...@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