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

No Display value for GNSS #90

Open
dwarning opened this issue Aug 17, 2023 · 1 comment
Open

No Display value for GNSS #90

dwarning opened this issue Aug 17, 2023 · 1 comment

Comments

@dwarning
Copy link
Collaborator

Hi Ronan,
your commit e3f344e stops the display for LAT and LON on my MN 100-2.
It is not the introduction of GLL - it is the change:

-    if (c == 13)
+    if ((c == 13) || (c == 10))

Going back to the former state everything is OK: Display and correct OpenCPN via NMEA and WiFi Adapter.

@Rodemfr
Copy link
Owner

Rodemfr commented Jan 3, 2024

I finally have a bit of time to work again on the project.

I added this patch to handle NMEA sentences from the shipdriver_pi plugin of OCPN which was not formatting the sentences correctly. But my implementation had a bug and shipdriver_pi has been fixed since.
So I implemented another way to check the end of a sentence by using the '*' character. It seems more robust that the CR/LF codes which are always bringing variability accross the systems.

You can test this implementation in branch "decode_full_rmc"

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

No branches or pull requests

2 participants