I was surprised to find that cartan_matrix has methods symmetrized_matrix and symmetrizer (see cartan_matrix.py lines 465 and 498) whereas CartanType only has symmetrizer (see cartan_type.py line 1686). Also I expected one of the two symmetrizer methods to call the other. In fact they both have their own code which are different implementations of the same algorithm.
This is not that important but does seem untidy. I am happy to tidy this up if there is a consensus on what (if anything) should be done. -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-combinat-devel+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/sage-combinat-devel/f50f9f52-769b-43ee-8a09-487fe483825bn%40googlegroups.com.
