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

Comment (by tscrim):

 There's some danger with calling it a Coxeter graph, as this was the first
 entry I got when Googling: http://en.wikipedia.org/wiki/Coxeter_graph. I
 don't believe this is what we want, and I believe the more common
 terminology is Coxeter diagram. This was my mistake in calling that method
 Coxeter graph, which I did without thinking/looking too much because it
 returns a `Graph` instance.

 +1 for refactoring out common code between Coxeter and Dynkin diagrams,
 but IMO it doesn't need to be done here.

--
Ticket URL: <http://trac.sagemath.org/ticket/17798#comment:23>
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