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 TTM sentence parse support #128

Closed Tapped closed 3 weeks ago

Tapped commented 3 months ago

https://github.com/AeroRust/nmea/issues/54 Add TTM sentence parse support

TTM is a stand-alone feature, but part of a new feature called "radar", when other radar messages as TLL or TLB is introduced.

Tapped commented 3 months ago

@elpiel Thank you, I fixed your suggestions!

elpiel commented 2 months ago

@elpiel Thank you, I fixed your suggestions!

I'm not sure if you are aware but the CI fails.

Tapped commented 3 weeks ago

@elpiel Yeah I noticed, but the errors seems to be unrelated to this PR.

I suspect its caused by an update of clippy. I'll make a separate PR to fix those errors, that I assume fails on latest main as well.

See for example: https://github.com/AeroRust/nmea/actions/runs/9164588924/job/25378071147?pr=128#step:5:65

Tapped commented 3 weeks ago

I'm not sure what to do with the codecov failing?

elpiel commented 3 weeks ago

Nothing, I will find some time to investigate it.

Thank you once again for your contributions and patience!

If you are looking for more fun on the nmea crate, there are a few key features that will make the crate very useful for embedded platforms.