#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 |
-------------------------------------+-------------------------------------
Changes (by jipilab):
* cc: nthiery (added)
Comment:
Hi,
While having a look at the diff of the current patch, I saw that in
CoxeterMatrixGroup the method coxeter_graph is renamed by coxeter_diagram.
I'm having thoughts about this for a while.
I would be tempted to have a clear distinction between dynkin diagram and
coxeter graphs. Especially considering #16126 coming.
I guess eventually, there could be a common class on top of dynkin
diagrams and coxeter graphs from which they inherit. But right now, it
makes more sense to me to try to keep the convention coxeter graph vs
dynkin diagram.
What do you think?
--
Ticket URL: <http://trac.sagemath.org/ticket/17798#comment:22>
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.
