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

Problems programatically switching modes with widgets #338

Closed
mramato opened this issue Nov 20, 2012 · 3 comments · Fixed by #407
Closed

Problems programatically switching modes with widgets #338

mramato opened this issue Nov 20, 2012 · 3 comments · Fixed by #407

Comments

@mramato
Copy link
Contributor

mramato commented Nov 20, 2012

Load the "simple CZML demo" in Sandcastle here Change czmlSensors to the below (the only addition is the view2D line) run it and hit the sensors button. crash.

    function czmlSensors(widget) {
        Sandcastle.declare(czmlSensors);    // For highlighting in Sandcastle.
        widget.removeAllCzml();
        widget.viewHome();
        widget.view2D.onClick();
        widget.loadCzml(gallery + 'LotsOfSensors.czml');
    }
@ghost ghost assigned bagnell Nov 20, 2012
@mramato
Copy link
Contributor Author

mramato commented Nov 20, 2012

@bagnell This also happens in your camera branch. If it's an easy fix, can we get it into master soonish?

@mramato
Copy link
Contributor Author

mramato commented Nov 21, 2012

This may be another manifestation of #319

@mramato
Copy link
Contributor Author

mramato commented Dec 21, 2012

Fixed in above pull.

@mramato mramato closed this as completed Dec 21, 2012
mramato added a commit that referenced this issue Jan 3, 2013
1. Handle case where cartesian is at or near the center of the ellipsoid.
2. For positions at the center, we now return `undefined`.
3. For positions near the center, we now return the result instead of getting stuck in an infinite loop.
4. Fixes #338 by handling case 2 during a morph.
5. Minor cleanup related to above changes.

CC @kring @bagnell
pjcozzi added a commit that referenced this issue Aug 9, 2015
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.

2 participants