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

Fix all GIS models #172

Closed
31 tasks done
EwoutH opened this issue Aug 25, 2024 · 6 comments
Closed
31 tasks done

Fix all GIS models #172

EwoutH opened this issue Aug 25, 2024 · 6 comments

Comments

@EwoutH
Copy link
Member

EwoutH commented Aug 25, 2024

We just started testing our GIS models in CI (see #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

  • GeoSchelling
    • Fix missing file: data/nuts_rg_60M_2013_lvl_2.geojson
    • Fix missing Model initialization
    • UserWarning: The weights matrix is not fully connected:
  • GeoSchellingPoints
    • Fix missing file: data/nuts_rg_60M_2013_lvl_2.geojson
  • Rainfall
    • Fix missing file: /vsigzip/data/elevation.asc.gz
    • Fix missing Model initialization
    • RasterLayer doesn't have access to Model (mesa-geo#236)
  • Population
    • Fix missing file: /vsizip/data/clip.zip
    • Fix missing Model initialization
    • RasterLayer doesn't have access to Model (mesa-geo#236)
  • AgentsAndNetworks
    • Provide the following required arguments (6312ebe):
      • campus
      • data_crs
      • buildings_file
      • walkway_file
      • lakes_file
      • rivers_file
      • driveway_file
      • num_commuters
    • unhashable type error (f24626e)
    • set output_dir (679b677)
  • GeoSir
    • Fix missing file: data/TorontoNeighbourhoods.geojson
    • Fix missing Model initialization
  • UrbanGrowth
    • Provide missing argument for RasterLayer.__init__(): model

PRs to fix any of these issues are very welcome!

Update:

@EwoutH
Copy link
Member Author

EwoutH commented Aug 25, 2024

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

@EwoutH
Copy link
Member Author

EwoutH commented Aug 25, 2024

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
Copy link
Member Author

EwoutH commented Sep 2, 2024

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

class AgentsAndNetworks(mesa.Model):

Or maybe load them from:

@EwoutH
Copy link
Member Author

EwoutH commented Sep 3, 2024

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
Copy link
Member Author

EwoutH commented Sep 3, 2024

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!

@EwoutH EwoutH closed this as completed Sep 3, 2024
@wang-boyu
Copy link
Member

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

@EwoutH EwoutH unpinned this issue Sep 3, 2024
@EwoutH EwoutH added the gis label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants