hydrologie / xhydro / 21261225132

93%

Build Type

push

github

Committed by web-flow

Commit Message

Address multiple warnings (#403)

<!-- Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [x] This PR addresses an already opened issue (for bug fixes /
features)
  - This PR fixes #xyz
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [x] If changes affect the GIS, Raven Modelling, Extremes.jl or Use
Case Example notebooks, they have been re-run (ReadTheDocs will not
update these).
- [x] If text has changed, ``make initialize-translations`` /
``make-translations.bat`` has been run and translations have been
updated.
- [x] (If applicable) Tests have been added.
- [x] CHANGELOG.rst has been updated (with summary of main changes).
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added.

### What kind of change does this PR introduce?

* Reduces the number of warnings from ~3000 to 30 (most of which coming
from xHydro itself, on purpose, until the next release).
  * Many small fixes to address upcoming changes to xarray/pandas.
* `stackstac` was generating multiple `gdal/rasterio` warnings, but has
had no updates since August 2024. The library has been replaced with
`odc-stac`, which is more up-to-date and does the same thing.
* Added an additional `xfail` to the tests in the GIS module to prevent
fails when the PlanetaryComputer servers are down.

### Does this PR introduce a breaking change?

* Moving from `stackstac` to `odc-stac` has changed the results
slightly, due to differences in how they manage projections.
* In `watershed_properties`, renamed `Upstream Area (sq. km).` to
`Upstream Area (sq. km)` (without the final dot).

### Other information:

Run Details

44 of 46 new or added lines in 7 files covered. (95.65%)

1 existing line in 1 file now uncovered.

3044 of 3290 relevant lines covered (92.52%)

4.94 hits per line