Update mesa.examples models to use scenario#3363
Merged
Conversation
|
Performance benchmarks:
|
EwoutH
approved these changes
Mar 8, 2026
Member
EwoutH
left a comment
EwoutH
left a comment
There was a problem hiding this comment.
Two checks, otherwise looks good so pre-approving!
|
|
||
| n: int = 50 | ||
| mean: float = 0.5 | ||
| std_dev: float = 0.2 |
Member
There was a problem hiding this comment.
You changed std_dev from 0.1 to 0.2, was that a conscious change?
| width: int = 50 | ||
| height: int = 50 | ||
| activation_order: Literal["Sequential", "Random", "Simultaneous"] = "Random" | ||
| payoff: None | dict[tuple[str, str], float] = None |
Member
There was a problem hiding this comment.
You changed payoffs to payoff, was this on purpose?
Member
Author
There was a problem hiding this comment.
It was not deliberate. The object in question is known as the payoff matrix in most literature so I just went with that without checking the original code
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the examples to use a custom scenario class. It also updates app.py and the tests.
I left out Sugarscape since there is a PR open already that makes the change.