Raster overlay UI/UX enhancements

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

Raster overlay UI/UX enhancements #649

Closed corybarr closed 7 months ago

corybarr commented 7 months ago

Adds custom UI widgets for CesiumPolygonRasterOverlayPrimand CesiumIonRasterOverlayPrim
Adds "Invert Selection" to CesiumPolygonRasterOverlayPrim
Makes CesiumPolygonRasterOverlayPrimand CesiumIonRasterOverlayPrim "alpha" attribute a slider
Makes BasisCurves created via Quick-Add menu have appropriate settings for type, wrap, and points attributes.
Sets ordering of UI groups to match Unreal (polygons->invert->rendering->credits)
Constrains cartographic polygon bindings to only be BasisCurves

Resolves #633 Resolves #618 Resolves #631

lilleyse commented 7 months ago

This doesn't need to be addressed now...

If I set a polygon overlay's render method to overlay it appears red. Is this something we have control over, or are we just rendering the pixel color exactly as it comes from native?

corybarr commented 7 months ago

This doesn't need to be addressed now...

If I set a polygon overlay's render method to overlay it appears red. Is this something we have control over, or are we just rendering the pixel color exactly as it comes from native?

We don't have control over it right now, but it's not a major lift. I created a ticket for it. #652

r-veenstra commented 7 months ago

Invert selection appears to obey the initial setting on load, but when changing the bool and refreshing the tileset it does not update accordingly. If I save the USDA with the updated bool and reload the stage, it works as expected.
Default values for the cartographic polygons appear to be good
Changing Invert Selection should probably trigger a tileset refresh (similar to the refresh that occurs when adding or removing polygons from the raster overlay). Though that could be frustrating unless https://github.com/CesiumGS/cesium-omniverse/issues/650 is fixed first

You should be able to test the invert issue with the following zip

TilesetClipping.zip

r-veenstra commented 7 months ago

Thanks, invert issues appear resolved now!

r-veenstra commented 7 months ago

I added basis_curves.GetCurveVertexCountsAttr().Set([4]) to the creation of basis curves, as without this property the curves never render, and the control points never show when trying to edit them