#13700: fix non-crystallographic "example" of Dynkin diagram
-----------------------------------------------+---------------------------
Reporter: hthomas | Owner: sage-
| combinat
Type: defect | Status: new
Priority: minor | Milestone: sage-6.4
Component: combinatorics | Resolution:
Keywords: root system, Dynkin diagram | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
-----------------------------------------------+---------------------------
Comment (by tscrim):
Actually, I'm thinking we should remove this outright:
{{{
sage: DynkinDiagram(['D',4]).an_instance()
Dynkin diagram of rank 3
}}}
I understand the point, but it is strange having it as a method of an
instance of a Dynkin diagram. I know this is a fault of
Python/`@staticmethod`, but still, if anything we would likely be better
served having this as a method attached to the function `DynkinDiagram`.
We could switch to `samples` as we have for `CartanType`(`Factory`).
Although switching to C,,3,, is good to me.
--
Ticket URL: <https://trac.sagemath.org/ticket/13700#comment:10>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
