Results 158 comments of Frédéric Chapoton

where does the merge conflict comes from ? which pull request ?

maybe the choice of the name `ComplexReflectionGroup` is a bit confusing with a category.

no, ok, I was just asking. It is indeed reasonable to keep your proposed name, with appropriate documentation

There is a typo "Klien" instead of Klein. For the name, I would suggest to keep it shorter. What about "ComplexReflectionG" ?

right. :smile: What about `ShephardToddFamilyGroup` in the class and `ShephardToddFamily` in the catalog ?

There remains an annoying issue about `latex` methods. This needs to be investigated.

What kind of change would you expect ? I am trying to make so that nothing changes, except that we get rid of the `Algebra` class inheritance. This means that...

I cannot investigate now. What I did was - change to Parent - see that tests in latex repr were failing - fix them as I could. I have no...

ah, great find. I had not seen that the loop variable was modified in the loop.. Eurk... What should we do ? Move back to the C-style for loop ?