Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test examples #65

Merged
merged 3 commits into from
Jan 11, 2024
Merged

Test examples #65

merged 3 commits into from
Jan 11, 2024

Conversation

Corvince
Copy link
Contributor

@Corvince Corvince commented Nov 1, 2023

Added a test that initiales and runs each model for 10 steps. Currently only handles core examples, not geo examples.

To make the test file run I had to modify some imports to be relative imports.

Otherwise this PR fixes 1 error and 1 warning:

In the epstein_civil_violence model the count_cops method was missing

The hexgrid examples used HexGrid, which is deprecated and should be changed to HexGridSingle.

@rht
Copy link
Contributor

rht commented Jan 10, 2024

Ruff check didn't pass.

@rht rht merged commit 385b804 into main Jan 11, 2024
3 checks passed
@rht rht deleted the test_examples branch January 11, 2024 07:13
@EwoutH
Copy link
Member

EwoutH commented Jan 11, 2024

Very useful, thanks!

And now somehow we need to link this to the main Mesa CI

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.

3 participants