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

Fix updating feature id set index #665

Closed lilleyse closed 7 months ago

lilleyse commented 7 months ago

Fixes a crash related to https://github.com/CesiumGS/cesium-omniverse/pull/644.

When copying Fabric materials it should use the original material path instead of the _materialSource path since changed property notifications come from the original material, not the _materialSource (which seems to be removed). Not doing this causes this assert to be hit.

To reproduce:

On this branch it won't crash. But you might still see artifacts related to https://github.com/CesiumGS/cesium-omniverse/issues/581.