-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
GotoSaturn binding not working #210
Comments
Thank you for the very clear issue report! It looks like the code that looks up "Saturn" gets confused because this image in our database has exactly that name as well. I think we should be able to fix this with a data update that renames the image. It might make sense to remove the image altogether since it looks like it might not really map onto the sky in a sensible way, anyway. |
pkgw
added a commit
to pkgw/wwt-core-catalogs
that referenced
this issue
Apr 29, 2022
These were just named "Saturn", causing problems for the `GotoSaturn` command functionality as reported in WorldWideTelescope/wwt-windows-client#210. I'm pretty sure that a simple rename should fix the issue. The astrometry for the "Infrared Ring Around Saturn" looks to be ballpark OK -- the top half of the image seems consistent with PS1-3pi, although with a substantial shift. "Big Band of Dust" doesn't agree astrometrically and it is mostly non-sky data, so we archive it.
Hi,
Peter, as the creator of this image I agree that this image should probably just be removed… I don’t think it even had WCS metadata embedded so am surprised it showed up at all.
Since this was a solar system observation, its coordinate only makes sense with respect to Saturn’s location at the time of observation, and only from Spitzer’s viewing geometry in space at that time… so not really meaningful as a sky overlay.
Cheers,
Robert
On Apr 29, 2022, at 6:44 AM, Peter Williams ***@***.******@***.***>> wrote:
Thank you for the very clear issue report! It looks like the code that looks up "Saturn" gets confused because this image in our database has exactly that name as well. I think we should be able to fix this with a data update that renames the image. It might make sense to remove the image altogether since it looks like it might not really map onto the sky in a sensible way, anyway.
—
Reply to this email directly, view it on GitHub<#210 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADX2F5TG7L4ZBKKBW3S72E3VHPRVRANCNFSM5UU4AQBQ>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
Robert L. Hurt, Ph.D (he/him) | Visualization Scientist
Caltech/IPAC | IPAC Communications and Education
Mail Code 314-6 | Pasadena CA 91125
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When I use the gotoSaturn binding, either with a view-button or through a midi controller, I get a switch to skyMode and an overlay with Saturn's dust clouds from Spitzer in stead. I'm using 6.1.1.0 and have tried this on both a windows 10 and a windows 11 (both 64bit)
The text was updated successfully, but these errors were encountered: