#15375: Extended Affine Weyl Groups SD40
-------------------------------------+-------------------------------------
       Reporter:  bump               |        Owner:  bump
           Type:  enhancement        |       Status:  needs_work
       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                |  dde0d4e57f9473f99c8428f378c8a1fd46c23325
         Branch:  public/15375       |     Stopgaps:
   Dependencies:  #10963, #14102     |
-------------------------------------+-------------------------------------

Comment (by nthiery):

 Replying to [comment:96 bump]:
 > 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.

 My current bet is that this ticket somehow forces
 `sage.sets.cartesian_product` to be loaded upon Sage's startup. Hence,
 when running `load_submodules(sage.sets)` it does not appear anymore
 as the first entry, and instead the output starts with the next one
 which turns out to be `sage.sets.real_set`.

 If that's confirmed, a good approach would be to make sure that all
 the affine Weyl group code is lazy imported. That's a good property to
 have anyway.

 Cheers,
                                       Nicolas

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