bluerobotics / bluerobotics/ping-cpp

Ping360 example uses auto messages without checking protocol support

Offen
#65 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
19
Forks
17
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Ping-protocol v1.1 introduced support for [auto transmission](https://docs.bluerobotics.com/ping-protocol/pingmessage-ping360/#2602-auto_transmit) for the ping360 message set, which [our example](https://github.com/bluerobotics/ping-cpp/blob/master/test/test-device-ping360.cpp#L56) uses without checking that the device is capable of handling and sending the relevant messages.

Ideally it should check the protocol version and fall back to only manual messages if relevant.
[Raised/discussed here](https://discuss.bluerobotics.com/t/obtaining-raw-data-from-the-ping360-using-ping-cpp/19573/2).

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start in test/test-device-ping360.cpp at the linked example and review the Ping-protocol v1.1 auto-transmission documentation. Add a protocol-capability check so unsupported devices use only manual messages, then verify the example's behavior for devices that do and do not support the relevant messages.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
api, embedded-iot
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.