hydromt icon indicating copy to clipboard operation
hydromt copied to clipboard

Compatibility with rio-vrt v.0.3.1

Open Tjalling-dejong opened this issue 10 months ago • 1 comments

Kind of request

Changing existing functionality

Enhancement Description

Currently the the test_to_xyz_tiles will break when version 0.3.1 of rio-vrt is installed. This is because the test wants to create virtual raster from a single raster. Rio-vrt v0.3.1 does not allow that.

I suggest to include a check in RasterDataArray.to_xyz_tiles that checks if for the given zoom level a single raster tile is given for creating a vrt and then either raises an error or issues a warning and does not continue to build a vrt.

Use case

No response

Additional Context

No response

Tjalling-dejong avatar Feb 12 '25 15:02 Tjalling-dejong

@roeldegoede Is the raster functionality to_xyz_tiles and to_slippy_tiles used in the FloodAdapt ecosystem to your knowledge? Otherwise we might as well remove these methods.

DirkEilander avatar Feb 18 '25 09:02 DirkEilander

I would love to see these functions being moved to another library or directly in the plugin where they are used. Our new goals for HydroMT are to be minimal in the number of utilities in the core. So if @roeldegoede is the only user with one of the plugins, let's first move it there. And if other people want it, we can also decide to move it to cht_tiling, for example.

deltamarnix avatar Mar 18 '25 15:03 deltamarnix

I don't think this functionallity is actively used somewhere and might already have been replaced by cht_tiling. So we could remove it from hydromt v1.

roeldegoede avatar Mar 26 '25 09:03 roeldegoede