#18722: Fix weight of Nakajima monomials
-------------------------------------+-------------------------------------
Reporter: tscrim | Owner: sage-combinat
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.8
Component: combinatorics | Resolution:
Keywords: crystals, | Merged in:
nakajima monomials | Reviewers:
Authors: Travis Scrimshaw | Work issues:
Report Upstream: N/A | Commit:
Branch: | f02f80cb660f9f35df0eba94eb1cdcf767deb22d
public/crystals/fix_nakajima_monomials-18722| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by bsalisbury1):
The following example failed with the original commit:
{{{
sage: M = crystals.NakajimaMonomials(['A',2,1],
RootSystem(['A',2,1]).weight_lattice(extended=True).fundamental_weight(0))
sage: m = M.module_generators[0]
sage: m.weight()
Lambda[0]
}}}
A conditional was added to {{{__classcall_private__}}} of
{{{CrystalsOfNakajimaMonomials}}} to ensure the weight lattice being used
is the extended weight lattice with the underlying Cartan type is affine.
Some doctests needed to be updated so that the input used the extended
weight lattice in affine examples.
Doctests passed on my machine, so if you agree with these changes, then
you may set the ticket to positive review on my behalf.
--
Ticket URL: <http://trac.sagemath.org/ticket/18722#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 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.