Mirror geoutils API changes

GlacioHack / xdem

Analysis of digital elevation models (DEMs)

https://xdem.readthedocs.io/

MIT License

123 stars 37 forks source link

Closed adehecq closed 4 months ago

adehecq commented 5 months ago

EDIT from @rhugonnet: This PR updates xDEM to work with GeoUtils v0.1.

In details:

Removes all "dst_" prefixes in Raster.reproject,
Replaces crop and shift to be not inplace by default,
Replaces argument of Raster.to_points "subset" with "sample",
Changes dst_vcrs to vcrs and src_vcrs to force_source_vcrs in DEM.to_vcrs,
Pins GeoUtils 0.1.*.

rhugonnet commented 5 months ago

Some issues related to GeoPandas/Pandas, allowing Pandas 2 while pinning to avoid these errors in: https://github.com/GlacioHack/geoutils/pull/481

rhugonnet commented 5 months ago

Tests passing locally after my last commit, hopefully also in CI!

rhugonnet commented 5 months ago

Waiting to see if we move forward with sample or subsample before merging!

rhugonnet commented 4 months ago

All tests passing with GeoUtils v0.1, merging!