#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.

Reply via email to