Skip to content

Improve-12-17-2025#240

Closed
rgleason wants to merge 91 commits into
masterfrom
improve
Closed

Improve-12-17-2025#240
rgleason wants to merge 91 commits into
masterfrom
improve

Conversation

@rgleason
Copy link
Copy Markdown
Owner

No description provided.

stelian42 and others added 30 commits September 20, 2025 20:58
Also, save DR coordinates into Sights.xml
- widen lunar altitude fields
- make pressure precision a hundreth, temperature precision a tenth
- fix duplicate formula in Calculate output
Retain 2 decimal digits for eye height as well
Thanks to OpenCPN provided fromDMM_Plugin() function, all degrees-like
measurements (heights, longitudes, latitudes, etc) can now be entered
in multiple formats like:
- 37°54.204' N
- N37 54 12
- 37°54'12"
- 37.9034
- 122°18.621' W
- 122w 18 37
 -122.31035

Some fields have also been changed to print the measurement is
standardized DD° MM.MMMM' format instead of a double value.
@rgleason
Copy link
Copy Markdown
Owner Author

I've build and run this locally. It looks good.

@rgleason
Copy link
Copy Markdown
Owner Author

@copilot Where to review?

Copy link
Copy Markdown

Copilot AI commented Dec 17, 2025

@rgleason I've opened a new pull request, #241, to work on those changes. Once the pull request is ready, I'll request review from you.

Celestial Navigation Plugin: Lunar Distance, Sight Enhancements, and Platform Support
@rgleason
Copy link
Copy Markdown
Owner Author

@stelian42 It won't let me merge without someone's review and approval other than me. I can override it but would rather not. Can you review it?

@rgleason
Copy link
Copy Markdown
Owner Author

Oh I guess copilot did the review and then when I reviewed that, the PR was merged into master. Closing.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants