AeroRust / nmea

NMEA 0183 - for communication between marine electronics such as echo sounder, sonars, anemometer, gyrocompass, autopilot, GNSS receivers and many other types of instruments. Defined and controlled by the National Marine Electronics Association (NMEA)
https://crates.io/crates/nmea
Other
57 stars 41 forks source link

Adds MWV and MDA parsers #63

Closed AndrewLipscomb closed 1 year ago

AndrewLipscomb commented 1 year ago

Adds a pair of parsers for WIMDA and WIMWV. No GPS Fix related messaging parsing for this set as they are both weather station based messages.

elpiel commented 1 year ago

Thank you for working on this! I will make sure to review the PR in the coming days.

codecov[bot] commented 1 year ago

Codecov Report

Base: 79.50% // Head: 79.50% // No change to project coverage :thumbsup:

Coverage data is based on head (379d7a2) compared to base (379d7a2). Patch has no changes to coverable lines.

:exclamation: Current head 379d7a2 differs from pull request most recent head a50f483. Consider uploading reports for the commit a50f483 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #63 +/- ## ======================================= Coverage 79.50% 79.50% ======================================= Files 22 22 Lines 854 854 ======================================= Hits 679 679 Misses 175 175 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

AndrewLipscomb commented 1 year ago

Alrighty - still getting the hang of Rust's docs and linting, but cargo fmt and cargo clippy seem like happy bois now. Added the test case too

Edit - forgot the pub members on MDA - fixed now. Give it a whirl Shirl