#6588: Categories for root systems
--------------------------------------------------------+-------------------
   Reporter:  nthiery                                   |          Owner:  
mhansen                 
       Type:  enhancement                               |         Status:  
needs_review            
   Priority:  major                                     |      Milestone:  
sage-5.0                
  Component:  combinatorics                             |       Keywords:  root 
systems, categories
Work_issues:                                            |       Upstream:  N/A  
                   
   Reviewer:  Anne Schilling, Mark Shimozono, PatchBot  |         Author:  
Nicolas M. ThiƩry       
     Merged:                                            |   Dependencies:  
#10817                  
--------------------------------------------------------+-------------------
Changes (by nthiery):

  * status:  needs_work => needs_review


Comment:

 Replying to [comment:13 davidloeffler]:
 > Fails doctests on Sage 5.0.beta7:
 > {{{
 > sage -t -long devel/sage-main/sage/combinat/root_system/weight_space.py
 > **********************************************************************
 > File "/storage/masiao/sage-5.0.beta7/devel/sage-
 main/sage/combinat/root_system/weight_space.py", line 126:
 >     sage: Q.null_root(0)[index]
 > Exception raised:
 >     Traceback (most recent call last):
 >       File "/storage/masiao/sage-5.0.beta7/local/bin/ncadoctest.py",
 line 1231, in run_one_test
 >         self.run_one_example(test, example, filename, compileflags)
 >       File "/storage/masiao/sage-5.0.beta7/local/bin/sagedoctest.py",
 line 38, in run_one_example
 >         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
 >       File "/storage/masiao/sage-5.0.beta7/local/bin/ncadoctest.py",
 line 1172, in run_one_example
 >         compileflags, 1) in test.globs
 >       File "<doctest __main__.example_1[17]>", line 1, in <module>
 >         Q.null_root(Integer(0))[index]###line 126:
 >     sage: Q.null_root(0)[index]
 >     TypeError: __call__() takes exactly 0 positional arguments (1 given)
 > **********************************************************************
 > }}}
 > (The patchbot found this, but I also reproduced it by hand)

 Sorry, already fixed on the Sage-Combinat queue where the review is
 occuring. I'll delete the patch here for the moment so that others don't
 waste time reviewing an outdated version.

 Thanks though for the report!

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