#16954: Game Theory: Build class for normal form games as well as ability to
obtain
Nash equilibria
-------------------------------------+-------------------------------------
Reporter: vinceknight | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.4
Component: game theory | Resolution:
Keywords: Game Theory, | Merged in:
Normal Form Games | Reviewers: Karl-Dieter Crisman
Authors: Vince Knight, | Work issues:
James Campbell | Commit:
Report Upstream: N/A | 986fa93705b9d790c8051abb65f87bbf53d98b17
Branch: | Stopgaps:
u/vinceknight/normal_form_games |
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by vinceknight):
* commit: dea6ac3c8f3801c97ec018ad5ee7d2152bd3eaf4 =>
986fa93705b9d790c8051abb65f87bbf53d98b17
* branch: u/vinceknight/nearing_the_finish_line =>
u/vinceknight/normal_form_games
Comment:
Replying to [comment:48 kcrisman]:
> > This is a weird one: the test doesn't fail for me as is but if I throw
in `Graphics object consisting of 2 graphics primitives` I then get:
> That is probably because you have not based it on the most recent Sage
release candidate. Trust me on this one.
Done and rebuilding Sage to catch up with the latest develop branch.
> > > * In built doc, apparently {{{`i`th}}} disagrees with Sage's
Sphinxification and gives it indigestion. Changing it to {{{`i` th}}}
fixes things, but I get that this might not be desirable. Unfortunately,
{{{:math:`i`th}}} doesn't give what we want either.
> >
> > Have changed the wording so we don't have an issue.
> There is another spot in the next line.
Apologies, have again changed the wording.
> > > * Also, apparently lazy importing `PIPE` was indeed what caused it
to be built for me in the documentation.
> >
> > To clarify: I'm still not getting anything to do with `PIPE` so is
this all ok or should I investigate and attempt to fix?
> Just to avoid the possibility (perhaps again it's something to do with
me using the latest rc), I would remove the lazy import and then import it
just the one place it is needed below. That worked for me.
I've done that (hopefully in the right spot: in the method that calls it).
>
> Otherwise the changes seem fine.
Great: please let me know if there's anything else :) (I guess I let you
modify the ticket `positive review`).
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=d6c570069dfba3eeb2f77baeee8b979f92a5fe9d
d6c5700]||{{{Adding test for graphics object}}}||
||[http://git.sagemath.org/sage.git/commit/?id=c500a661a3978eb5714a52a120c42ffcaa15967c
c500a66]||{{{Fixing `ith` issue}}}||
||[http://git.sagemath.org/sage.git/commit/?id=8fed12ea334f99d86e8d8f88736e28f9c7869164
8fed12e]||{{{Adding optional test tag}}}||
||[http://git.sagemath.org/sage.git/commit/?id=986fa93705b9d790c8051abb65f87bbf53d98b17
986fa93]||{{{Changing how PIPE and POpen is imported}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/16954#comment:49>
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.
