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

GPS/GNSS NMEA Sentence Processor Library needs updating #555

Open
djaus2 opened this issue Oct 18, 2023 · 1 comment
Open

GPS/GNSS NMEA Sentence Processor Library needs updating #555

djaus2 opened this issue Oct 18, 2023 · 1 comment
Assignees
Labels
p1 update required Existing docs need to be updated to the latest APIs

Comments

@djaus2
Copy link

djaus2 commented Oct 18, 2023

GPS/GNSS NMEA Sentence Processor Library sample code
Main ussue is that Meadow.Foundation.Sensors.Location.Gnss.NmeaParsing is now depracated, as per Nuget documentation.
Add Nuget package Meadow.Foundation.Sensors.Location.Gnss to the app.
Modify line 9 to

using Meadow.Foundation.Sensors.Location.Gnss; //.NmeaParsing;

NmeaSentenceProcessor is part of that class

Note for what follows I was targetting Project Labs V3,

  • Usual change of public class MeadowApp : App<F7Micro, MeadowApp> to:
    public class MeadowApp : App<F7CoreComputeV2>
  • Change void Initialize() to
    public override Task Initialize()
  • Change Device.SerialPortNames.Com4, to
    Device.PlatformOS.GetSerialPortName("COM1"), or whatever COM port is used. COM1 is the one in mikroBUS No. 1
  • After serialPort.MessageReceived line add the Baud Rate setting eg:
    serialPort.BaudRate = 9600;
  • At end of Initialize() insert:
    return base.Initialize();
@patridge patridge added the update required Existing docs need to be updated to the latest APIs label Jun 16, 2024
@patridge patridge added the p1 label Jun 24, 2024
@patridge
Copy link
Collaborator

@duduita Do you mind reviewing this to see if there is still an issue we need to resolve.

The NuGet package mentioned doesn't mention being deprecated here.

https://www.nuget.org/packages/Meadow.Foundation.Sensors.Location.Gnss.NmeaProcessor/

And it is the only package I find with that prefix: Meadow.Foundation.Sensors.Location.Gnss.

We do have a broken link to the class API docs at the top, though: NmeaSentenceProcessor name points to a 404 link. I found that class here instead, which somewhat aligns with the namespace issue mentioned originally: https://developer.wildernesslabs.co/docs/api/Meadow.Foundation/Meadow.Foundation.Sensors.Location.Gnss/NmeaSentenceProcessor/.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p1 update required Existing docs need to be updated to the latest APIs
Projects
None yet
Development

No branches or pull requests

3 participants