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

fix: APB nextpoint paths, XTE in nautical miles #68

Merged
merged 1 commit into from
May 8, 2022

Conversation

sergei
Copy link
Contributor

@sergei sergei commented Mar 24, 2021

  • Fixes APB sentence not emitted #61 by using navigation.courseGreatCircle.nextPoint.bearingTrue and navigation.courseGreatCircle.nextPoint.bearingMagnetic individually
  • Change units of XTE to NM
  • Make all degrees positive

Change units of XTE to NM
Make all degrees positive
@tkurki
Copy link
Member

tkurki commented May 8, 2022

This is in line with.

I would have preferred 2 commits, one for XTE and another for the APB fix, but what the heck, we've ignored this for way too long, apologies, so all I can say is thank you.

@tkurki tkurki changed the title Fix GH-61 fix: APB nextpoint paths, XTE in nautical miles May 8, 2022
@tkurki tkurki merged commit d6fbc22 into SignalK:master May 8, 2022
@tkurki tkurki added the fix label May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

APB sentence not emitted
2 participants