add WmsRasterOverlay support and UI

CesiumGS / cesium-omniverse

Bringing the 3D geospatial ecosystem to Omniverse

https://cesium.com/platform/cesium-for-omniverse/

Apache License 2.0

55 stars 7 forks source link

add WmsRasterOverlay support and UI #670

Closed corybarr closed 6 months ago

corybarr commented 7 months ago

Adds Web Map Service (WMS) support:

Test file: WmsTest.zip

Resolves #609

corybarr commented 7 months ago

@r-veenstra I guess on good max ranges on the zoom level. Can you see if they're reasonable?

corybarr commented 7 months ago

@r-veenstra I'm wondering if adding a "Refresh" button to the WMS raster overlay is the best way to handle users' property updates.

r-veenstra commented 7 months ago

I've done some testing, seems to be working as expected. Comments below.

Clicking the Refresh Tiles button on a tileset or Reload All Tilesets in the Cesium Debug window doesn't result in the WMS overlay updating based on latest values - should it?
We probably want to include these settings too:

r-veenstra commented 7 months ago

@r-veenstra I guess on good max ranges on the zoom level. Can you see if they're reasonable?

The defaults in there look okay and are consistent with Unreal too

lilleyse commented 7 months ago

Clicking the Refresh Tiles button on a tileset or Reload All Tilesets in the Cesium Debug window doesn't result in the WMS overlay updating based on latest values - should it?

Tilesets and raster overlays are now separate things in the C++ layer so OmniTileset would need to explicitly reload the OmniRasterOverlays that it references. @corybarr could you make that fix?

corybarr commented 6 months ago

We probably want to include these settings too:

@r-veenstra This is an issue with all raster overlays. I created #675 for this.

corybarr commented 6 months ago

Clicking the Refresh Tiles button on a tileset or Reload All Tilesets in the Cesium Debug window doesn't result in the WMS overlay updating based on latest values - should it?

Tilesets and raster overlays are now separate things in the C++ layer so OmniTileset would need to explicitly reload the OmniRasterOverlays that it references. @corybarr could you make that fix?

@lilleyse Added.