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

Concurrent morphs cause crash #377

Closed
mramato opened this issue Dec 12, 2012 · 5 comments
Closed

Concurrent morphs cause crash #377

mramato opened this issue Dec 12, 2012 · 5 comments

Comments

@mramato
Copy link
Contributor

mramato commented Dec 12, 2012

Easy to reproduce with two widgets examples

  1. Hit the 2D Map button on the top example
  2. While the top widget is morphing, hit the 2D map button on the bottom example
  3. When the top morph finishes, the example will crash.

I'm pretty sure this is a bug that occurs in any situation where multiple morphs are going at once across different scenes.

@mramato
Copy link
Contributor Author

mramato commented Mar 29, 2013

I spent some time debugging this problem. The issue is that SceneTransitioner adds and removes tweens inside the callback of other tweens. This is a big no-no because we end up modifying an array that is currently being iterated inside of TWEEN.update. Given the number of other problems with SceneTransitioner (like #602), I'm going to leave this as-is until it either becomes a larger problem or we finally refactor SceneTransitioner into its final form.

@mramato
Copy link
Contributor Author

mramato commented May 2, 2013

Do to change in transitioner architecture in #725, this is no longer an issue.

@mramato mramato closed this as completed May 2, 2013
@mramato
Copy link
Contributor Author

mramato commented May 2, 2013

I was wrong, this is still an issue.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jun 11, 2014

@mramato can you still reproduce this?

In Sandcastle:

HTML:

<style>
    @import url(../templates/bucket.css);
</style>
<div id="cesiumContainer" class="top"></div>
<div id="cesiumContainer2" class="bottom"></div>

JavaScript:

var viewer = new Cesium.Viewer('cesiumContainer');
viewer.sceneModePicker.viewModel.duration = 30000.0;
var viewer2 = new Cesium.Viewer('cesiumContainer2');
viewer2.sceneModePicker.viewModel.duration = 30000.0;

@mramato
Copy link
Contributor Author

mramato commented Jun 19, 2014

I think the original problem is gone; there's still a crash but it's not related to having two widgets, I'll write up a separate issue.

@mramato mramato closed this as completed Jun 19, 2014
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