#8649: to_tableau method broken for crystals of type B
------------------------------+---------------------------------------------
Reporter: aschilling | Owner: sage-combinat
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.4
Component: combinatorics | Keywords: crystals
Author: Anne Schilling | Upstream: N/A
Reviewer: Dan Bump | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Changes (by nthiery):
* reviewer: => Dan Bump
Comment:
Ah, we did a simultaneous review :-) Well, for the record, here were my
comments:
All test pass on 4.3.5, and the changes are technically good (crystals are
far more coercion friendly now). Thanks Anne for handling this!
I am just not very comfortable with the special casing with 0 to handle
type B, which sounds more like a workaround than a mathematically
meaningful rule.
Dan: what do you think? If that's fine with you, please set a positive
review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8649#comment:3>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
