search

corteva / rioxarray

geospatial xarray extension powered by rasterio
https://corteva.github.io/rioxarray
Other
511 stars 81 forks source link

RasterioDeprecationWarning in clip_box() #551

Closed jonasViehweger closed 2 years ago

jonasViehweger commented 2 years ago

https://github.com/corteva/rioxarray/blob/21f2ec9aebc2c7e5223cc484e8dd4eacb26f01e0/rioxarray/raster_array.py#L772-L773

Connected to https://github.com/rasterio/rasterio/issues/2248.

Using rio.clip_box() throws this deprecation warning:

RasterioDeprecationWarning: The height, width, and precision parameters are unused, deprecated, and will be removed in 2.0.0.

This should be fixed by simply removing those parameters.

(I wanted to submit this as a pull request because it's such an easy fix, but following the contributing guidelines the pre-commit hooks throw this error at me:

[INFO] This may take a few minutes... An unexpected error has occurred: CalledProcessError: command: ('/home/jonas/miniconda3/envs/rioxarray/bin/python3.1', '-mvirtualenv', '/home/jonas/.cache/pre-commit/repog8c5a1hh/py_env-python3.1', '-p', 'python3.1') return code: 1 expected return code: 0 stdout: RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.1'

stderr: (none)

)

jonasViehweger commented 2 years ago

I don't know how essential the pre commit hooks are, but I just submitted a pull request with the changes commited without running the hooks: https://github.com/corteva/rioxarray/pull/552