Improve tests in `test_affine` once core class restructured with `BiasCorr` parameters

GlacioHack / xdem

Analysis of digital elevation models (DEMs)

https://xdem.readthedocs.io/

MIT License

123 stars 37 forks source link

Improve tests in `test_affine` once core class restructured with `BiasCorr` parameters #485

Open rhugonnet opened 4 months ago

rhugonnet commented 4 months ago

The logic in test_biascorr can be copied to test all types of inputs (raster-raster, point-raster, point-point), but for efficiency we should only do this once the function class parameters are fixed (fit optimizer, binning, etc) to avoid rewriting the tests twice. See #435.

rhugonnet commented 4 months ago

And in more details, some other to-do in those classes once restructured:

[ ] Homogenize tests in test_affine,
[ ] Add point-point tests in test_base.
[ ] Add _fit_pts_pts for ICP and related test in test_affine,
[ ] Check that #422 and #423 are resolved (probably fixed by #480).