CesiumGS / cesium-omniverse

Bringing the 3D geospatial ecosystem to Omniverse
https://cesium.com/platform/cesium-for-omniverse/
Apache License 2.0
51 stars 7 forks source link

Add Tile Map Service (TMS) and Web Map Tile Service (WMTS) raster overlay support #691

Closed corybarr closed 5 months ago

corybarr commented 5 months ago

(Draft until UI polishing is complete)


Adds TMS support:

image

Adds WMTS support:

image


UI/UX Notes:

image

This PR hides them instead. I didn't find an obvious way to gray them out.

While Omniverse supports string[] parameters, they aren't by default editable in the UI: image

In this PR, users supply the strings comma-separated.

r-veenstra commented 5 months ago

This appears to be functioning as expected.

My only question is should there be some feedback in the console when WMTS is configured incorrectly?

For example, with WMS and TMS if I put in a wrong URL, I'll usually get some form of error.

However, with WMTS nothing displays in the log at all.

corybarr commented 5 months ago

This appears to be functioning as expected.

My only question is should there be some feedback in the console when WMTS is configured incorrectly?

For example, with WMS and TMS if I put in a wrong URL, I'll usually get some form of error.

However, with WMTS nothing displays in the log at all.

I think it would be better. I'll look into it.

For ref, it seems that a valid, although empty, result is returned when WMTS Urls are bad: image

A bad TMS Url will fail the if (result) test and trigger the error logger.