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

Moving the camera eventually crashes (if you're unlucky). #6783

Closed
OmarShehata opened this issue Jul 7, 2018 · 8 comments · Fixed by #7667
Closed

Moving the camera eventually crashes (if you're unlucky). #6783

OmarShehata opened this issue Jul 7, 2018 · 8 comments · Fixed by #7667

Comments

@OmarShehata
Copy link
Contributor

Brought up in this forum thread.

Try holding ctrl and rotating with left mouse click in this Sandcastle. If you get the camera in just the right position (the origin), you'll get:

An error occurred while rendering.  Rendering has stopped.
undefined
DeveloperError: normalized result is not a number
Error
    at new DeveloperError (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:540:19)
    at Function.Cartesian3.normalize (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:8132:19)
    at rotateVertical (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:186217:28)
    at Camera.rotateUp (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:186208:9)
    at rotate3D (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:231295:20)
    at tilt3DOnEllipsoid (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:231534:9)
    at tilt3D (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:231476:13)
    at reactToInput (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:230287:21)
    at update3D (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:231807:9)
    at ScreenSpaceCameraController.update (https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:231840:13)

You can see a video here (I'm just moving my mouse in clockwise in circles and holding ctrl).

Just putting the camera at the origin will also trigger this:

var viewer = new Cesium.Viewer('cesiumContainer');
viewer.camera.setView({
    destination : new Cesium.Cartesian3(),
});
@mmacros
Copy link

mmacros commented Jul 24, 2018

Also if you move the camera fast, looking underground and above ground, the renderer crash, showing the following error.

captura

@OmarShehata
Copy link
Contributor Author

Came up on the forum here.

@esparano
Copy link

esparano commented Dec 12, 2018

I've been seeing this problem for about 2 years now, actually. I can reproduce it pretty reliably. Here's the updated line numbers for developers:

image

I'm doing the same thing Omar is doing, in the Sandcastle demo app. Holding middle mouse button and rotating the camera like crazy.

@just-a-user-or-something

I am seeing the same stack as @esparano:

Developer Error: normalized result is not a number
normalize
rotateVertical
rotateUp
rotate3D
tilt3DOnEllipsoid
tilt3D
reactToInput
update3D

@hpinkos
Copy link
Contributor

hpinkos commented Mar 12, 2019

Also reported by @mholthausen in #7585

@OmarShehata
Copy link
Contributor Author

Reported on the forum today as well.

@OmarShehata
Copy link
Contributor Author

From the forum posted mentioned above:

Hi Omar,
I found another commercial website, John Wayne airport, using Cesium and had similar runtime error. You don't have to do any out of ordinary. You can just keep zooming with the mouse wheel until the camera passes the ground then try to look up to the sky from that position, the app crashed.
http://jwaflighttrackviewer.airportnetwork.com/
I think this issue needs to be elevated to be fixed ASAP.
Thanks for your wonderful contribution to the Cesium community!
Hoaininh

@cesium-concierge
Copy link

Congratulations on closing the issue! I found these Cesium forum links in the comments above:

https://groups.google.com/d/msg/cesium-dev/ku9822o-t8I/iEqGNEPdBgAJ
https://groups.google.com/d/msg/cesium-dev/c6-oUcHFUik/yBSWhdmFAgAJ

If this issue affects any of these threads, please post a comment like the following:

The issue at #6783 has just been closed and may resolve your issue. Look for the change in the next stable release of Cesium or get it now in the master branch on GitHub https://github.com/AnalyticalGraphicsInc/cesium.

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

Successfully merging a pull request may close this issue.

6 participants