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

Satellites plugin rounds observer elevation to 1000 meters #2848

Closed
rkinnett opened this issue Nov 16, 2022 · 5 comments
Closed

Satellites plugin rounds observer elevation to 1000 meters #2848

rkinnett opened this issue Nov 16, 2022 · 5 comments
Assignees
Labels
bug Something likely wrong in the code importance: low Small problem, rarely visible, no crash
Milestone

Comments

@rkinnett
Copy link

The Satellites plugin appears to round the user's location elevation setting to the nearest 1000 meters which is very coarse. For low-Earth orbiting satellites at mid-elevation, this yields roughly 0.1 deg pointing error.

To reproduce, pause Stellarium, select a fast-moving LEO satellite, then change site elevation gradually between 0 to 999m and observe no change in satellite position, then increment to 1000m and observe discrete jump of roughly 0.1 deg, then no change between 1000 and 1999m and another jump at 2000m, etc.

Please consider rounding to nearest 1m or 10m or 100m instead of 1000m.

This will improve usefulness of Stellarium for planning observations of satellite transits of celestial objects, although it is understood that there are other limitations on satellite position accuracy.

@rkinnett rkinnett changed the title Satellites plugin rounds or floors observer location elevation to 1000 meters Satellites plugin rounds observer location elevation to 1000 meters Nov 16, 2022
@rkinnett rkinnett changed the title Satellites plugin rounds observer location elevation to 1000 meters Satellites plugin rounds observer elevation to 1000 meters Nov 16, 2022
@gzotti gzotti self-assigned this Nov 16, 2022
@gzotti gzotti added this to the 1.2 milestone Nov 16, 2022
@gzotti gzotti closed this as completed in b8d34ec Nov 16, 2022
@gzotti
Copy link
Member

gzotti commented Nov 16, 2022

Thanks for noting this!

@gzotti gzotti added bug Something likely wrong in the code importance: low Small problem, rarely visible, no crash labels Nov 17, 2022
@rkinnett
Copy link
Author

Thanks for quick resolution!

@alex-w alex-w added the state: published The fix has been published for testing in weekly binary package label Nov 20, 2022
@github-actions
Copy link

Hello @rkinnett!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Dec 25, 2022
@github-actions
Copy link

Hello @rkinnett!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

@rkinnett
Copy link
Author

Confirmed working in Stellarium v1.2. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something likely wrong in the code importance: low Small problem, rarely visible, no crash
Development

No branches or pull requests

3 participants