#13391: WeylCharacterRing improvement
------------------------------------------+---------------------------------
       Reporter:  bump                    |         Owner:  bump          
           Type:  enhancement             |        Status:  needs_work    
       Priority:  major                   |     Milestone:  sage-5.4      
      Component:  combinatorics           |    Resolution:                
       Keywords:  WeylCharacterRing, Lie  |   Work issues:                
Report Upstream:  N/A                     |     Reviewers:  Anne Schilling
        Authors:  Daniel Bump             |     Merged in:                
   Dependencies:                          |      Stopgaps:                
------------------------------------------+---------------------------------
Changes (by jdemeyer):

  * status:  positive_review => needs_work


Comment:

 1) There are issues with the documentation:
 {{{
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeylCharacterRing.dot_reduce:7:
 WARNING: Inline interpreted text or phrase reference start-string without
 end-string.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeightRing:0: WARNING: Block
 quote ends without a blank line; unexpected unindent.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeylCharacterRing.dot_reduce:11:
 WARNING: Inline interpreted text or phrase reference start-string without
 end-string.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeightRing:2: WARNING:
 Definition list ends without a blank line; unexpected unindent.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeylCharacterRing.dot_reduce:14:
 WARNING: Inline interpreted text or phrase reference start-string without
 end-string.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeightRing:5: WARNING: Block
 quote ends without a blank line; unexpected unindent.
 dochtml.log:/release/merger/sage-5.4.beta0/local/lib/python2.7/site-
 packages/sage/combinat/root_system/weyl_characters.py:docstring of
 sage.combinat.root_system.weyl_characters.WeightRing:4: ERROR: Unexpected
 indentation.
 }}}

 2) The patch needs a proper commit message.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13391#comment:16>
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.

Reply via email to