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

add Direction of Crosshairs in Oculars #4075

Closed
wants to merge 1 commit into from

Conversation

ultrapre
Copy link
Contributor

Description

Add directional indicators to the eyepiece view when the crosshairs are enabled and aligned with the equatorial coordinate system. This will help observers quickly identify the directional angles of celestial objects in the eyepiece. Such indicators are also widely used in sketches of deep-sky objects.

Screenshots (if appropriate):

image

image

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update
  • Housekeeping

How Has This Been Tested?

Enable oculars, enable crosshairs, check align crosshairs.

Switch flip options.

Test Configuration:

  • Operating system: Windows 11 21H2
  • Graphics Card: GTX 1060

@alex-w
Copy link
Member

alex-w commented Jan 13, 2025

The current master:

stellarium-014

@ultrapre
Copy link
Contributor Author

Ok, is it an existing function?

@alex-w
Copy link
Member

alex-w commented Jan 13, 2025

Ok, is it an existing function?

Yes, I've added it many years ago (a compass rose)

@ultrapre ultrapre closed this Jan 13, 2025
@ultrapre ultrapre deleted the ocular_direction branch January 13, 2025 16:24
@gzotti
Copy link
Member

gzotti commented Jan 13, 2025

(Yes, I was sure we have this already...)

@sunshuwei
Copy link
Contributor

微信图片_20250114020405

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

Successfully merging this pull request may close these issues.

4 participants