Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Model clipped inappropriately #6451

Closed
emackey opened this issue Apr 16, 2018 · 2 comments
Closed

Model clipped inappropriately #6451

emackey opened this issue Apr 16, 2018 · 2 comments

Comments

@emackey
Copy link
Contributor

emackey commented Apr 16, 2018

While experimenting with lighting options for a glTF model provided in KhronosGroup/COLLADA2GLTF#172, I discovered the model sometimes disappears from view while moving the camera. This didn't start happening until I merged master into my branch, bringing in logDepth along with it.

Git bisect isolates a specific commit from LogDepth where the problem is introduced: 0d90d98

The original model from that issue has some un-related validation errors, here's a cleaned-up copy of the model:

Collada2gltf_issue172.zip

And the reproduce code:

var viewer = new Cesium.Viewer('cesiumContainer', {
    infoBox : false,
    selectionIndicator : false,
    shadows : false,
});

function createModel(url) {
    viewer.entities.removeAll();

    var position = Cesium.Cartesian3.fromDegrees(-123.074085, 44.050050, 10);
    
    var heading = Cesium.Math.toRadians(-20);
    var pitch = 0;
    var roll = 0;
    var hpr = new Cesium.HeadingPitchRoll(heading, pitch, roll);
    var orientation = Cesium.Transforms.headingPitchRollQuaternion(position, hpr);

    var entity = viewer.entities.add({
        name : url,
        position : position,
        orientation : orientation,
        model : {
            scale: 10,
            uri : url,
            minimumPixelSize : 128,
            maximumScale : 20000
        }
    });
    viewer.trackedEntity = entity;
}

createModel('../../SampleData/models/emackey/Collada2gltf_issue172.gltf');

Here's the bad behavior:

logdepthclipissue

@emackey
Copy link
Contributor Author

emackey commented Apr 16, 2018

Note that this bug is "not yet released", since it was introduced with LogDepth #5851 last week.

@mramato
Copy link
Contributor

mramato commented Apr 16, 2018

I tagged this for next release since it's a bad regression. CC @bagnell @lilleyse

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants