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.

Sign up for GitHub

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

Jump to bottom

Zooming to dynamic geometry broken #6269

Closed
mramato opened this issue Feb 24, 2018 · 2 comments
Closed

Zooming to dynamic geometry broken #6269

mramato opened this issue Feb 24, 2018 · 2 comments
Assignees
@hpinkos
Labels
priority - next release

Comments

@mramato
Copy link
Member

mramato commented Feb 24, 2018

@hpinkos check out this demo: http://localhost:8080/Apps/Sandcastle/index.html?src=CZML%20Custom%20Properties.html&label=CZML It no longer zooms to the geometry, I think this was a recent regression with all of the cleanup work.
@mramato
Copy link
Member Author

mramato commented Feb 24, 2018

Looks like it's broken on cesiumjs.org as well, so this might have been introduced last release? Maybe the zoomTo changes for 3D Tileset?

@hpinkos
Copy link
Contributor

hpinkos commented Feb 26, 2018

@ggetz can you look into this one too? It looks like it might have broken with your CPU usage changes. I think I tracked it down to this commit: d698876

@ggetz ggetz mentioned this issue Feb 27, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hpinkos hpinkos

Labels
priority - next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mramato @hpinkos