Skip to content
New issue

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

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

Already on GitHub? # to your account

Tilted CV view and adjust transition from 2D to CV #4501

Merged
merged 6 commits into from
Jan 2, 2017
Merged

Tilted CV view and adjust transition from 2D to CV #4501

merged 6 commits into from
Jan 2, 2017

Conversation

ggetz
Copy link
Contributor

@ggetz ggetz commented Oct 21, 2016

Fixes #3878.

Columbus view now has a tilt and the animation from 2D to CV has been updated.

Based on the PR by @JoshuaStorm in #3983. I followed his camera setup for CV, but updated the transition to be smoother and crashes no longer happen when transitioning.

There is a slight jump in the transition from CV back to 2D, but that is the current behavior in master. I believe it is from the FOV switch from perspective to orthographic.

@pjcozzi
Copy link
Contributor

pjcozzi commented Oct 21, 2016

@bagnell can you please review?

@ggetz update CHANGES.md.

@ggetz
Copy link
Contributor Author

ggetz commented Oct 21, 2016

Updated.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 2, 2017

@ggetz can you please merge in master and I'll test and merge?

@ggetz
Copy link
Contributor Author

ggetz commented Jan 2, 2017

@pjcozzi Merged!

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 2, 2017

@ggetz did you mean to delete your update to CHANGES.md? I will update it.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 2, 2017

The new morph is good with me.

@pjcozzi pjcozzi merged commit 6da36fb into CesiumGS:master Jan 2, 2017
@pjcozzi pjcozzi deleted the cv-tilt branch January 2, 2017 16:14
@ggetz
Copy link
Contributor Author

ggetz commented Jan 2, 2017

did you mean to delete your update to CHANGES.md? I will update it.

No, that was an accident.

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

Successfully merging this pull request may close these issues.

2 participants