Invertable classification with alpha #5755
Conversation
|
@bagnell thanks for the pull request! I noticed that I am a bot who helps you make Cesium awesome! Thanks again. |
|
At first glance names seem OK. Are you sure this should be a scene-level setting? Perhaps we need a way to composite a primitives/tilesets into a micro pass that has these settings (which also answers your derived commands question and might also be able to fix #5676)? |
| })); | ||
|
|
||
| var classification = viewer.scene.primitives.add(new Cesium.Cesium3DTileset({ | ||
| url : 'http://localhost:8002/tilesets/meshTest/' |
There was a problem hiding this comment.
Can you put this - or something like this - on beta.cesium.com?
Source/Scene/FrameState.js
Outdated
| * The alpha of unclassified 3D Tile geometry when {@link FrameState#invertClassification} is <code>true</code>. | ||
| * @type {Number} | ||
| */ | ||
| this.invertClassificationAlpha = undefined; |
There was a problem hiding this comment.
Should this default to 1.0 instead?
There was a problem hiding this comment.
Or maybe not since only the Scene default matters...
…rag depth and depth textures are supported.
…Sandcastle example.
|
I'm merging this. What needs reviewing is now in #5836. |
Adds
Scene.invertClassificationandScene.invertClassificationAlpha. WhenScene.invertClassificationis set totrue, any 3D Tiles geometry that intersects classification volumes will be drawn opaque while geometry that does not intersect the volume will be drawn translucent with the alpha set toScene.invertClassificationAlpha.I'm open to different names for these properties.
@pjcozzi @lilleyse The derived commands approach doesn't seem very clean. Another option would be to add new passes.