Skip to content

10 show coordinates on screen #23

Merged
merged 9 commits into from
Oct 6, 2025
Merged

10 show coordinates on screen #23

merged 9 commits into from
Oct 6, 2025

Conversation

kleinc
Copy link
Member

@kleinc kleinc commented Oct 6, 2025

Description of change

Change camera coordinates to show different coordinates of interest, in an interpretable format:

  • Display camera coordinates in lat/lon format on screen
  • Display elevation range derived from the z - pivot coordinate
  • Elevation range only visible when in pivot mode

Closes #10

The text was blurry before, but should now be clear. Also added the showing of elevation for a target point
The code is now refactored to use the new coordinate system. It is also generalized more to not look as messy as before.
@kleinc kleinc self-assigned this Oct 6, 2025
@kleinc kleinc linked an issue Oct 6, 2025 that may be closed by this pull request
@gautegf
Copy link
Member

gautegf commented Oct 6, 2025

THe lat/lon need more decimals to be accurate

Copy link
Member

@franmagn franmagn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything good, left just a comment

Copy link
Member

@adriahso adriahso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Made some small comments.

Copy link
Member

@adriahso adriahso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@franmagn franmagn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMT!

@kleinc kleinc merged commit 56fa67c into dev Oct 6, 2025
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show coordinates on screen
4 participants