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
56 stars 41 forks source link

Add HDT sentence parser #82

Closed ekuinox closed 1 year ago

ekuinox commented 1 year ago

54 Implements parser for HDT sentence 😄

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 76.92% and project coverage change: +0.04 :tada:

Comparison is base (a95be8c) 79.20% compared to head (e0dfab2) 79.24%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #82 +/- ## ========================================== + Coverage 79.20% 79.24% +0.04% ========================================== Files 26 27 +1 Lines 1000 1012 +12 ========================================== + Hits 792 802 +10 - Misses 208 210 +2 ``` | [Impacted Files](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust) | Coverage Δ | | |---|---|---| | [src/lib.rs](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust#diff-c3JjL2xpYi5ycw==) | `0.00% <ø> (ø)` | | | [src/parser.rs](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust#diff-c3JjL3BhcnNlci5ycw==) | `77.82% <0.00%> (-0.31%)` | :arrow_down: | | [src/sentences/mod.rs](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust#diff-c3JjL3NlbnRlbmNlcy9tb2QucnM=) | `0.00% <ø> (ø)` | | | [src/sentences/hdt.rs](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust#diff-c3JjL3NlbnRlbmNlcy9oZHQucnM=) | `80.00% <80.00%> (ø)` | | | [src/parse.rs](https://codecov.io/gh/AeroRust/nmea/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AeroRust#diff-c3JjL3BhcnNlLnJz) | `71.42% <100.00%> (+0.76%)` | :arrow_up: | 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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

elpiel commented 1 year ago

Thank you very much for your PR :tada: !

I've just update criterion and set the new MSRV, so if you update with main and fix rustfmt you should be good to go!

https://github.com/AeroRust/nmea/commit/ab80227b7074b55f357aacf505a0816def6cc1f5

ekuinox commented 1 year ago

Thank you very much for your PR 🎉 !

I've just update criterion and set the new MSRV, so if you update with main and fix rustfmt you should be good to go!

ab80227

Thank you for reviewing. I merged main branch and fixed format!