projectmesa / mesa-examples

Seminal agent-based models developed using Mesa
Other
113 stars 133 forks source link

Fix all GIS models #172

Closed EwoutH closed 1 week ago

EwoutH commented 2 weeks ago

We just started testing our GIS models in CI (see https://github.com/projectmesa/mesa-examples/pull/171), and currently unfortunately all 7 models that are tested present one or multiple errors

The errors are visible in any recent CI run of the Test GIS models workflow. They can also be reproduced and tested locally by running the test_gis_examples.py script with pytest.

image

Current errors

PRs to fix any of these issues are very welcome!

Update:

EwoutH commented 2 weeks ago

(help is wanted on all our issues, including the ones without this label, so removing it)

EwoutH commented 2 weeks ago

Made a couple of fixes, now that #175 and #176 are merged the first two models pass, GeoSchellingPoints and GeoSir. For now I'm going to leave it at that.

Five to go!

EwoutH commented 1 week ago

@wang-boyu Could you add sensible default values to the AgentsAndNetworks model?

https://github.com/projectmesa/mesa-examples/blob/3d29e4c215154396e20a5a39a09b8a4043b22de6/gis/agents_and_networks/src/model/model.py#L44

Or maybe load them from:

https://github.com/projectmesa/mesa-examples/blob/3d29e4c215154396e20a5a39a09b8a4043b22de6/gis/agents_and_networks/scripts/run.py#L34

EwoutH commented 1 week ago

All warnings resolved! The next thing that needs to happen, is the big backport to the mesa-2.x branch, to make sure they're there working, before we update them with Mesa 3.0 stuff.

EwoutH commented 1 week ago

The big backport was successful, working, CI tested GIS examples are now available both on the main and mesa-2.x branches.

Completed! Thanks @wang-boyu!

wang-boyu commented 1 week ago

GIS example tests were long overdue. Thanks for your work on this!