pico-examples icon indicating copy to clipboard operation
pico-examples copied to clipboard

feat: add GPS UART module with NMEA parsing and checksum validation

Open yousefyasser opened this issue 1 year ago • 8 comments

  • Added checksum validation for incoming NMEA strings
  • Added parsing of NMEA strings of type GPRMC to extract needed information, such as: latitude and longitude

Co-authored-by: @RasheedAtia Co-authored-by: @seifAbbas

yousefyasser avatar Jan 18 '25 20:01 yousefyasser

There doesn't appear to be a main() function here which calls uart_gps_init or process_gps_data() ? :confused: The examples in pico-examples generally include at least one "standalone example" that creates a UF2 file that can be flashed to a Pico and actually "does something". If this is purely "library code" rather than a "standalone executable example", perhaps https://github.com/raspberrypi/pico-extras would be more suitable?

lurch avatar Jan 21 '25 12:01 lurch

Hello @lurch, ok I will add the main function right now

yousefyasser avatar Jan 21 '25 12:01 yousefyasser

Hello @lurch, ok I will add the main function right now

Thanks! Could you also please clarify which licence this code is released under?

lurch avatar Jan 21 '25 13:01 lurch

Hello @lurch, ok I will add the main function right now

Thanks! Could you also please clarify which licence this code is released under?

Sure, this code is released under the MIT license

yousefyasser avatar Jan 21 '25 13:01 yousefyasser

Sure, this code is released under the MIT license

A comment in the source-code saying that (for the benefit of future readers) would be great.

lurch avatar Jan 21 '25 13:01 lurch

What hardware are you using to test this? I've only seen i2c gps modules so far (I haven't looked too hard).

peterharperuk avatar Jan 21 '25 14:01 peterharperuk

What hardware are you using to test this? I've only seen i2c gps modules so far (I haven't looked too hard).

I used the Neo-6M GPS module.

yousefyasser avatar Jan 21 '25 14:01 yousefyasser

Too many printfs - you can't see the results for all the output

peterharperuk avatar Feb 13 '25 19:02 peterharperuk