Amaury Dehecq
Amaury Dehecq
The issue is caused by the fact that when downsampling, we are rounding up the shape ([here](https://github.com/GlacioHack/geoutils/blob/bb88ed665098cc131bf66bcec692a8701c65744e/geoutils/raster/raster.py#L487)). So when reading the data a few lines below in `_load_rio`, an out_shape...
Improve docstrings of `Raster.value_at_coords` and `Raster.interp_points`, or merge the two methods.
Another point: The docstring of Raster.interp_point regarding argument `input_latlon` is not clear enough. It's unclear whether the input coordinates should be given as (lon, lat) or (lat, lon). It currently...
That's a good point. Yes we should ensure that both the Raster and Vector class have a minimum set of attributes.
Great idea !
Hi @jhkennedy, yes, I was thinking about submitting a PR, but I was working on the conda install directly, not on a clone of the repository. I tried cloning the...
A few comments: - to open the data, the best is to use `geoutils.rasters.load_multiple_rasters` which optmize memory usage by first downloading only metadata, cropping and reprojecting only in areas of...
> I plan to revisit this matter after the 21/08/2024 meeting. Is there anything new to discuss here?
Overall, I agree with adding functionalities to address certain scalar metrics. Just a few remarks/questions: - I believe this should sit in geoutils rather than xdem, as it can be...
I didn't know riff, but I think if we could reduce the number of linters to one that manages all, that would certainly make things easier. It's seems normal that...
My own remarks: - I agree, "dem_to_be_aligned" is a bit long. I was thinking of "source_dem" but I'm open to other terms (I saw the term "secondary" in other places,...