Addressing progress bar issue

perrygeo / python-rasterstats

Summary statistics of geospatial raster datasets based on vector geometries.

BSD 3-Clause "New" or "Revised" License

525 stars 117 forks source link

Closed jeronimoluza closed 1 week ago

jeronimoluza commented 2 weeks ago

Added tqdm to main.py in zonal_stats function
- stats generator is created doing gen_zonal_stats() and then the total length of the generator (for tqdm to know) is done doing total=sum(1 for _ in stats). Not sure how I feel about this.
- if progress == True, the zonal_stats() function will return the list comprehension using the tqdm module, if not it will return the original generator list-initialization.
Added tqdm in project.optional-dependencies in pyproject.toml.

jeronimoluza commented 1 week ago

Just added those changes, @perrygeo ! Let me know if there's anything else you need me to do.

perrygeo commented 1 week ago

The errors in CI are unrelated to this PR. Looks like a breaking change in Fiona. I'll deal with it another PR.

perrygeo commented 1 week ago