#15279: RootSystem __init__ builds the dual twice, breaking initialization of
non-
crystallographic root systems
----------------------------------------------+----------------------------
Reporter: darij | Owner:
Type: defect | Status:
Priority: major | positive_review
Component: combinatorics | Milestone: sage-5.13
Keywords: root-system, sage-combinat | Resolution:
Authors: Darij Grinberg | Merged in:
Report Upstream: N/A | Reviewers: Travis
Branch: | Scrimshaw
Dependencies: | Work issues:
| Commit:
| Stopgaps:
----------------------------------------------+----------------------------
Comment (by nthiery):
This sounds like a merge that went wrong, so +1 on the change. You might
want to add a test if this impacts non crystalographic root systems (yes
we want to progressively support them!!!).
In the long run, we would want something better than just trying and
hoping for the best. We should be able to tell exactly when the dual
should be defined or not.
--
Ticket URL: <http://trac.sagemath.org/ticket/15279#comment:4>
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/groups/opt_out.
