#15375: Extended Affine Weyl Groups SD40
-------------------------------------+-------------------------------------
       Reporter:  bump               |        Owner:  bump
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.8
      Component:  combinatorics      |   Resolution:
       Keywords:  days54, coxeter,   |    Merged in:
  days64, days65                     |    Reviewers:  Dan Bump, Anne
        Authors:  Daniel Bump, Dan   |  Schilling
  Orr, Anne Schilling, Mark          |  Work issues:
  Shimozono, Nicolas Thiery.         |       Commit:
Report Upstream:  N/A                |  2b78b7bdc2db6835dadfad978e5f6992f0546649
         Branch:                     |     Stopgaps:
  public/combinat/extended_affine_weyl_groups-15375|
   Dependencies:  #10963, #14102     |
-------------------------------------+-------------------------------------

Comment (by bump):

 I looked at the patchbot shortlogs for the first three machines reporting
 build failure. They all report the same error:

 {{{
 OSError: [combinat ] /home/worker/sage-patchbot/local/lib/python2.7/site-
 packages/sage/combinat/root_system/extended_affine_weyl_group.py:docstring
 of
 
sage.combinat.root_system.extended_affine_weyl_group.ExtendedAffineWeylGroup_Class.WF_to_PW0_func:11:
 ERROR: Content block expected for the "WARNING" directive; none found.
 }}}

 I think the "Since this is used ..." at line 1068 needs to be indented.
 Other bots are reporting a doctest error in dev_tools.py.

 {{{
 **********************************************************************
 File "src/sage/misc/dev_tools.py", line 459, in
 sage.misc.dev_tools.import_statements
 Failed example:
     load_submodules(sage.sets)
 Expected:
     load sage.sets.cartesian_product... succeeded
     load sage.sets.set_from_iterator... succeeded
 Got:
     load sage.sets.real_set... succeeded
     load sage.sets.set_from_iterator... succeeded
 **********************************************************************
 }}}
 I suspect this is unrelated to this patch. It may be caused by #18553
 which was merged three weeks ago and which touched sage.sets.real_set.py.

 However I think the above docstring needs to be fixed and since Nicolas
 has some comments I'm changing the status back to needs_work.

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