opened
shizejin
username_0
Add some games in "test_pure_nash.jl".

1. Matching Pennies Game with 0 Pure Action Nash equilibrium;
2. Coordination game with 2 Pure Action Nash equilibria;
3. Unanimity Game with more than two players.

In the Unanimity Game, I use `combinations` from `Combinatorics`, and thus `Combinatorics` is required.
Add some complicated games for testing "pure_nash".
issue
created
cc7768
username_1
Hi @username_0. Thanks for adding these tests!

It is very useful to test the edge case that has 0 pure action nash equilibrium and a more complicated game. This should help keep us safe against breaking these functions in the future.

I don't see anything I would change before merging. If @username_2 agrees then it is ready to be merged.
comment
created
oyamad
username_2
Looks good to me.
comment
created
jstac
username_3
@username_0 Looks good, many thanks.
comment