perrygeo / python-rasterstats

Summary statistics of geospatial raster datasets based on vector geometries.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Lock Fiona version to prevent breaking change

ericboucher opened this issue · comments

Describe the bug
The recent upgrade of Fiona to 1.9.0 breaks the geojson_out feature which then returns fiona.model objects instead of GeoJSON dicts. We faced this issue in https://github.com/WFP-VAM/prism-app/ and fixed it with WFP-VAM/prism-app#730

To Reproduce

Steps to reproduce the behavior:

  1. How did you install rasterstats and its dependencies?
    Installing through docker and requirements.txt, installing version 0.17.0
  2. What datasets are necessary to reproduce the bug? Please provide links to example data if necessary.
    NA
  3. What code is necessary to reproduce the bug? Provide the code directly below or provide links to it.
    installing rasterstats and running zonal_stats with geojson_out=True

In the repo PRISM listed above, before the fix, running the tests would highlight the error.

Pinning the Fiona dependency version to 1.8.22 should be enough to fix this for now, before diving into the underlying issue.

PS - Thanks for this awesome lib @perrygeo !