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

Crash when switching scene modes during flight #4570

Open
mramato opened this issue Nov 1, 2016 · 5 comments
Open

Crash when switching scene modes during flight #4570

mramato opened this issue Nov 1, 2016 · 5 comments

Comments

@mramato
Copy link
Contributor

mramato commented Nov 1, 2016

Found during release testing, but confirmed to happen as far back as 1.25 so I don't think it needs to hold it up. We should fix it for 1.28 however.

  1. Run Cesium Viewer and switch to 2D
  2. Zoom in anywhere.
  3. Press the home button and switch to 3D during the flight.
  4. Click anywhere to immediately finish the morph.
  5. Crash
TypeError: Unable to get property 'height' of undefined or null reference
TypeError: Unable to get property 'height' of undefined or null reference
   at Camera.prototype._updateCameraChanged (http://localhost:8080/Source/Scene/Camera.js:333:9)
   at Scene.prototype.initializeFrame (http://localhost:8080/Source/Scene/Scene.js:2338:9)
   at CesiumWidget.prototype.render (http://localhost:8080/Source/Widgets/CesiumWidget/CesiumWidget.js:680:13)
   at render (http://localhost:8080/Source/Widgets/CesiumWidget/CesiumWidget.js:72:25)
@hpinkos
Copy link
Contributor

hpinkos commented Nov 1, 2016

Duplicate #1596?

@mramato
Copy link
Contributor Author

mramato commented Nov 1, 2016

I don't think so. They may be related but this is a crash, not just stuff disappearing during the morph.

@tfili
Copy link
Contributor

tfili commented Dec 1, 2016

@mramato who needs to look at this? Is it @bagnell?

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 1, 2016

Pushing to 1.29 to not hold up the release.

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 19, 2016

Removing next release label since this is not as critical as others. Will relabel when I triage soon.

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

4 participants