GlacioHack / xdem

Analysis of digital elevation models (DEMs)
https://xdem.readthedocs.io/
MIT License
123 stars 37 forks source link

Clarify `resolution` in terrain docstrings #491

Closed rhugonnet closed 3 months ago

rhugonnet commented 3 months ago

This PR specifies that the argument resolution is optional in terrain functions, moves it down to the end of the list, and adds a note on aspect independence from resolution. Tests were adjusted to all pass the resolution argument instead of relying on it as second positional.

Resolves #490