disableDepthTestDistance causes weird mouse interaction

[hpinkos]

disableDepthTestDistance pushes the billboards to the near plane, which messes up the collision detection when you try to zoom in.

See discussion in #6802 (comment)

var viewer = new Cesium.Viewer('cesiumContainer', {
    terrainProvider: Cesium.createWorldTerrain()
});
viewer.scene.globe.depthTestAgainstTerrain = true;

var lon = -122.1958;
var lat = 46.1915;
for (var i = 0; i < 20; i++) {
    for (var j = 0; j < 20; j++) {
        viewer.entities.add({
            position : Cesium.Cartesian3.fromDegrees(lon + i * -0.001, lat + j * -0.001),
            billboard : {
                image : '../images/facility.gif',
                heightReference : Cesium.HeightReference.RELATIVE_TO_GROUND,
                verticalOrigin: Cesium.VerticalOrigin.BOTTOM,
                disableDepthTestDistance: Number.POSITIVE_INFINITY
            }
        });
        
        viewer.entities.add({
            position : Cesium.Cartesian3.fromDegrees(lon + i * 0.001, lat + j * 0.001),
            billboard : {
                image : '../images/facility.gif',
                heightReference : Cesium.HeightReference.RELATIVE_TO_GROUND,
                verticalOrigin: Cesium.VerticalOrigin.BOTTOM,
                disableDepthTestDistance: Number.POSITIVE_INFINITY
            }
        });
    }
}

viewer.camera.flyTo({
    destination: Cesium.Cartesian3.fromDegrees(lon, lat, 10000)
});

[OmarShehata]

Just to tie in the conversation from #8009, @raycrummey you mentioned your workaround was to implement a custom zoom function. In case it helps others blocked on this, did you simply disable the collision detection or did you do something else?

I tried viewer.scene.screenSpaceCameraController.enableCollisionDetection = false; but I can still see the issue there.

[raycrummey]

Omar,
One workaround is to disable Cesium zoom, this.viewer.scene.screenSpaceCameraController.enableZoom = false;
Then you could implement a scaled down version of handleZoom()

[benjaminmur]

Hi @raycrummey !

I would be interested in how did you replace the handleZoom() function of cesium by yours ?
Did you modify the code and build the version with your fix ?
Or can you replace the function in your own code by setting your own function in some way ?

Btw, can you show me your own function ?

Thanks a lot for your response.

[OmarShehata]

Came up again here #8476.

[lilleyse]

Opened #8810 for rendering billboards in a separate pass, which would help fix this problem.

[YVin3D]

Looks like this commit fixes this. @kring do you happen to know if the bug you were working on for that commit is related to this issue?

[kring]

Could be @YoussefV. That commit was for #8600, which fixed a bunch of depth-related problems.

[mramato]

If this is fixed, that means we can look into enabling the billboard unburying path by default, right @hpinkos?

[hpinkos]

@mramato #6838 needs to be fixed. And probably #7376 and #8588 too