#16331: Game Theory: Build capacity to solve matching games in to Sage.
-------------------------------------+-------------------------------------
       Reporter:  vinceknight        |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.4
      Component:  game theory        |   Resolution:
       Keywords:  Game Theory,       |    Merged in:
  Matching Games,                    |    Reviewers:
        Authors:                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  ceb3802118125503bd230aae717f2afd01e646c3
  
u/vinceknight/game_theory__build_capacity_to_solve_matching_games_in_to_sage_|  
   Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by tscrim):

 Some notes on the recent commits. Please remove the list brackets in the
 `all` statement so it doesn't build the full list, so:
 {{{
 while not all(s.partner for s in suitors):
 }}}
 Note in these timings, there is a failure at the first entry:
 {{{
 sage: L = range(10000)
 sage: %timeit all([x for x in L])
 1000 loops, best of 3: 758 µs per loop
 sage: %timeit all(x for x in L)
 100000 loops, best of 3: 3.02 µs per loop
 }}}

 The added doctests are formatted wrong. The tests themselves need one more
 level of indentation and it shoulds be `TESTS::` (mainly 2 colons).

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