search

CesiumGS / cesium

An open-source JavaScript library for world-class 3D globes and maps :earth_americas:
https://cesium.com/cesiumjs/
Apache License 2.0
13.03k stars 3.51k forks source link

Possible (minor) bug in Classification Sandcastle #12062

Closed javagl closed 4 months ago

javagl commented 4 months ago

When disabling "Show Classification" in https://sandcastle.cesium.com/?src=3D%20Tiles%20Photogrammetry%20Classification.html then it still shows semi-transparent red boxes around the tileset:

Cesium Classification Sandcastle

Despite the comment saying // The same b3dm tileset used for classification, but rendered normally for comparison.

Changing the alpha value to 0.0 fixes this, but if the semi-transparent box is intentional, then this issue can be closed quickly.

ggetz commented 4 months ago

Thanks @javagl! I believe they are intended to be transparent, though I'd be open to tweaking the comment for clarity.

javagl commented 4 months ago

OK, it wasn't entirely clear whether the semi-transparent bounding boxes had been intentional (and I just started digging into that whole "classification" topic on the implementation level, but probably have to skip more details here). Probably no important action here then.