#15456: fix bug in has_right/left_descents in Weyl group code
-------------------------+-------------------------------------------------
Reporter: | Owner:
zabrocki | Status: needs_review
Type: | Milestone: sage-6.2
defect | Resolution:
Priority: major | Merged in:
Component: | Reviewers:
combinatorics | Work issues:
Keywords: | Commit:
Authors: | d0bdccadb38c68a0019d2056daebfd601b8910cb
Frédéric Chapoton | Stopgaps:
Report Upstream: N/A |
Branch: |
u/chapoton/15456 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by chapoton):
Here is new (better) proposal:
The main idea is "please never override has_descent" and "please implement
either has_left_descent or has_right_descent" in every instance (if only
as a place holder raising `NotImplementedError`).
In `categories/coxeter_group`, there is only the mechanism:
* has_descent calls either has_left_descent or has right_descent
* they call each other
In instances of Coxeter group, one implements either has_left_descent or
has_right_descent or both. Even in a minimalistic way, raising a
`NotImplementedError`.
What do you think ?
--
Ticket URL: <http://trac.sagemath.org/ticket/15456#comment:17>
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.
