#17798: Create a class for Coxeter matrices and types
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.6
      Component:  group theory       |   Resolution:
       Keywords:  Coxeter, groups,   |    Merged in:
  matrices, types                    |    Reviewers:
        Authors:  Travis Scrimshaw   |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  e61a4f3927c1d096f8ece1138ed33abbd928594d
  public/combinat/coxeter_matrices-17798|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by darij):

 I fear I cannot be of much help with this ticket, as I don't even know the
 difference between a Coxeter and a Cartan type... But please simplify that
 `CoxeterMatrix.__init__`. It is so complicated it is not pythonic in any
 good sense of this word. Python's the weak type system should not keep us
 from writing constructors (e.g., `from_cartan_type`) tailored to specific
 input data. I have no idea why catch-all constructors have become a
 standard in Sage, but this kind of programming gave us #17124 and probably
 some more ugliness I don't remember.

--
Ticket URL: <http://trac.sagemath.org/ticket/17798#comment:6>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to