Skip to content

Draft update to make_event - #1153

Open
tturocy wants to merge 1 commit into
masterfrom
issue_1147
Open

tturocy wants to merge 1 commit into
masterfrom
issue_1147

Conversation

@tturocy

@tturocy tturocy commented Sep 18, 2026

Copy link
Copy Markdown
Member

This closes the inconsistency in make_event (which allowed setting of event probabilities which was not what was intended).

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@tturocy

tturocy commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

@rahulsavani @wyz2368 This is a first step towards one of the items found in the survey review. However, it leaves us with a problem: We do not have a way of converting from an information set (player move) to an event.

Within the current API as designed, I'm not sure where the best place for that to sit is in terms of operations. But it is definitely an operation we want as there are clear use cases - transforming a game by assuming one of the players is actually nonstrategic and plays a fixed behaviour.

@tturocy tturocy added this to the gambit-17.0.0 milestone Sep 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

make_event should require all probability distributions to be the same

1 participant