Skip to content
This repository has been archived by the owner on May 26, 2024. It is now read-only.

Add decoding support for RMC messages from Galileo, GLONASS, GNSS #22

Merged
merged 9 commits into from
Nov 9, 2020

Conversation

aentinger
Copy link
Member

@aentinger aentinger commented Nov 2, 2020

Hi @generationmake 👋 Can you please test with your receiver and let me know if those GNRMC messages are decoded successfully? I'd also love to have a GLRMC as well as a GARMC message so I can feed them into the unit test system. Can you provide those datasets?

This fixes #19.

@aentinger aentinger self-assigned this Nov 2, 2020
@github-actions
Copy link

github-actions bot commented Nov 2, 2020

Memory usage change @ d36f5ca

Board flash % RAM for global variables %
arduino:samd:mkrvidor4000 🔺 +304 - +304 +0.12 - +0.12 🔺 +4 - +4 +0.01 - +0.01
Click for full report table
Board examples/NMEA-Basic
flash
% examples/NMEA-Basic
RAM for global variables
%
arduino:samd:mkrvidor4000 304 0.12 4 0.01
Click for full report CSV
Board,examples/NMEA-Basic<br>flash,%,examples/NMEA-Basic<br>RAM for global variables,%
arduino:samd:mkrvidor4000,304,0.12,4,0.01

@107-systems 107-systems deleted a comment from github-actions bot Nov 2, 2020
@generationmake
Copy link

Hi @aentinger
decoding of GPS, GLONASS and GNRMC messages works and seems to be correct.
Unfortunately I couldn't get a fix on the GLONASS signal, so I couldn't record a message. I will do more tests on this.
Additionally I haven't yet configured a Galileo receiver. I will figure out how to do this and perform a test.

@aentinger
Copy link
Member Author

Good morning @generationmake 👋 I'll be merging this then since the RMC message for Galileo should have the exact same setup, I don't think there's a need right now to test it individually.

@generationmake
Copy link

HI @aentinger
I managed to get a GLONASS dataset of u-blox NEO-7M:
$GLRMC,185101.00,A,4838.00179,N,01301.60481,E,4.246,167.52,091120,,,A*76

I tested and it worked fine.
I also reviewed the code and it is OK to merge.

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

Successfully merging this pull request may close these issues.

Add support for GLONASS and concurrent receivers
2 participants