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

Bypass loading of clipped tiles #658

Closed corybarr closed 6 months ago

corybarr commented 7 months ago

Do not merge until https://github.com/CesiumGS/cesium-native/issues/803 is resolved.


Add the following to functionality to polygon raster overlays:

image

(It seems tooltips don't wrap if they go off the screen.)

Test file: AustrialiaBlockClipping.zip

lilleyse commented 7 months ago

@corybarr given Kevin's comment in https://github.com/CesiumGS/cesium-native/issues/803#issuecomment-1925986441, it may be possible to test this with another dataset like Melbourne Photogrammetry (in the asset depot).

corybarr commented 7 months ago

@corybarr given Kevin's comment in CesiumGS/cesium-native#803 (comment), it may be possible to test this with another dataset like Melbourne Photogrammetry (in the asset depot).

@lilleyse Good call. It works well and feels very snappy loading just this section:

image

Logs confirm it works.

corybarr commented 7 months ago

@corybarr given Kevin's comment in CesiumGS/cesium-native#803 (comment), it may be possible to test this with another dataset like Melbourne Photogrammetry (in the asset depot).

@lilleyse Good call. It works well and feels very snappy loading just this section:

image

Logs confirm it works.

Clipping test for Melbourne: MelbourneClippingTest.zip

lilleyse commented 7 months ago

@corybarr I tested that https://github.com/CesiumGS/cesium-native/pull/808 fixes the clipping issues with P3DT.

corybarr commented 6 months ago

@corybarr I tested that CesiumGS/cesium-native#808 fixes the clipping issues with P3DT.

Awesome. Will update and merge after that fix merged.

lilleyse commented 6 months ago

@corybarr could you update CHANGES.md?

corybarr commented 6 months ago

@corybarr could you update CHANGES.md?

@lilleyse pushed

lilleyse commented 6 months ago

@corybarr could you update cesium-native in this branch now that https://github.com/CesiumGS/cesium-native/pull/808 is merged?

corybarr commented 6 months ago

@corybarr could you update cesium-native in this branch now that CesiumGS/cesium-native#808 is merged?

cesium-native is updated. The fix is confirmed working: image