Opened 5 years ago

Last modified 3 years ago

#19034 needs_work enhancement

Game Theory: Converting Extensive form games to Normal form.

Reported by: FRWard Owned by:
Priority: major Milestone: sage-6.9
Component: game theory Keywords: game theory
Cc: hhlorrimore, jcambell, vinceknight Merged in:
Authors: Francis Rhys Ward Reviewers:
Report Upstream: N/A Work issues:
Branch: u/FRWard/EF2NFG (Commits) Commit: 91917d7bb3ca7048d64a3758be952b030f3852ac
Dependencies: #18945 Stopgaps:

Description

For the newly created extensive form game class implement a to_nfg method which converts an extensive form game to normal form.

Change History (7)

comment:1 Changed 5 years ago by vinceknight

  • Cc vinceknight added

comment:2 Changed 5 years ago by FRWard

  • Branch set to u/FRWard/EF2NFG
  • Commit set to 2067b56257c74b33e3c6c0ddc60983bfc790739e
  • Status changed from new to needs_review

Last 10 new commits:

​30c3989Merge branch 'extensiveform' of github.com:theref/sage-game-theory into extensiveform
​9a28bbbMinor things
​7634248Adding a reference to gambit
​d0b301fAnother question.
​19d800eresolving merge conflicts
​72f8a04Tidied up the documentation for 'build_utility_dictionary'.
​59e84c1Tidying up the `to_nfg` method.
​c94b28eAdding information about normal form game conversion to doc string.
​8464d7aSlight speed increase and changes to `build_utility_dictionary`.
​2067b56Adding my name to Authors.

comment:3 Changed 5 years ago by kdilks

I don't know exactly how this works, but most things in this ticket need to somehow be marked as optional, as they require the optional package 'gambit', and the documentation won't build/tests fail if you don't have Gambit installed.

comment:4 Changed 5 years ago by FRWard

Thanks, I'll look into that now.

comment:5 Changed 5 years ago by git

  • Commit changed from 2067b56257c74b33e3c6c0ddc60983bfc790739e to 91917d7bb3ca7048d64a3758be952b030f3852ac

Branch pushed to git repo; I updated commit sha1. New commits:

​91917d7adding optional tags

comment:6 Changed 5 years ago by FRWard

I think that should all be fixed now

comment:7 Changed 3 years ago by mderickx

  • Status changed from needs_review to needs_work

Needs to be rebased on sage8.1beta3

Note: See TracTickets for help on using tickets.