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

Using Camera.lookAt followed by camera.setView breaks mouse controls #7094

Open
slozier opened this issue Sep 28, 2018 · 1 comment
Open

Comments

@slozier
Copy link
Contributor

slozier commented Sep 28, 2018

It looks like by doing the following, the camera.position will be set to (0, 0, 0) and then trying to adjust the camera using the mouse will cause a failure:

viewer.camera.lookAt(Cesium.Cartesian3.fromDegrees(15, 0, 100), new Cesium.HeadingPitchRange(0, 0, 100));
viewer.camera.setView({destination: Cesium.Cartesian3.fromDegrees(15, 0, 100)});

// then try to adjust the camera using the mouse

Using sandcastle it gives this error:

DeveloperError: normalized result is not a number
DeveloperError@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:540:19
Cartesian3.normalize@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:8139:19
rotateVertical@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:188049:17
Camera.prototype.rotateUp@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:188040:9
rotate3D@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:233530:13
spin3D@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:233409:13
reactToInput@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:232513:21
update3D@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:234040:9
ScreenSpaceCameraController.prototype.update@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:234075:13
Scene.prototype.initializeFrame@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:239589:9 
CesiumWidget.prototype.render@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:250532:13
render@https://cesiumjs.org/Cesium/Build/CesiumUnminified/Cesium.js:249915:25
@hpinkos
Copy link
Contributor

hpinkos commented Sep 28, 2018

Thanks for reporting this @slozier!

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