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

GearboxAssy.gltf does not appear in the center of the screen in Cesium.js v1.47 #38

Closed
cx20 opened this issue Jul 3, 2018 · 6 comments

Comments

@cx20
Copy link
Owner

cx20 commented Jul 3, 2018

I tried to display GearboxAssy.gltf with Cesium.js v1.47.
However, it does not seem to be displayed in the center of the screen.

Cesium.js + GearboxAssy.gltf result:
https://cx20.github.io/gltf-test/examples/cesium/index.html?model=GearboxAssy
image
As Cesium.js v1.46 was displayed in the center of the screen, I think that the specification changed probably.
It is necessary to investigate how to display the glTF model in the center like before.

@emackey
Copy link
Contributor

emackey commented Jul 9, 2018

There's been some discussion that the bounding sphere calculation got messed up (possibly by me... oops). I don't have a link to that issue handy but it might be the cause.

@emackey
Copy link
Contributor

emackey commented Jul 13, 2018

This is almost certainly caused by CesiumGS/cesium#6784 and should be fixed in the next release.

@cx20
Copy link
Owner Author

cx20 commented Jul 13, 2018

@emackey Thank you for clarifying the cause. I would like to confirm the operation again at the next release of Cesium.js.

@lilleyse
Copy link

Yes this was fixed by CesiumGS/cesium#6784

I tested the code on https://cx20.github.io/gltf-test/examples/cesium/index.html?model=GearboxAssy in Cesium's Sandcastle in master and it now zooms to the model correctly.

@cx20
Copy link
Owner Author

cx20 commented Jul 18, 2018

@lilleyse Thank you for fix and checking.
I will do the test again when Cesium.js v1.48 is released and I think that if it does not matter, I will close this Issue.

@cx20
Copy link
Owner Author

cx20 commented Aug 1, 2018

I confirmed that this issue was solved with Cesium.js v1.48.
https://cx20.github.io/gltf-test/examples/cesium/index.html?model=GearboxAssy
image

@cx20 cx20 closed this as completed Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants