Omniscan 450: Cannot change speed of sound. Wrong message id.
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 25/100
Direzione di ricerca
Inizia da src/definitions/omniscan450.json nel repository ping-protocol collegato ed esamina la PR #221 di ping-protocol, poi verifica come la PR #170 di ping-python propaga la modifica. Il lavoro è completo quando l’operazione set_speed_of_sound di OMNISCAN450 usa l’ID messaggio 116 e il dispositivo non restituisce più un NACK.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
This is my first issue/PR on open-source code. I used AI to navigate the process but have manually reviewed to the best of my knowledge.
The core issue stems from the ping-protocol repository. Issue is opened there too.
Description
The set_speed_of_sound message for OMNISCAN450 sonar is defined with message ID 1002
in src/definitions/omniscan450.json, but the device expects ID 116.
This can be deducted from the documentation here: https://docs.ceruleansonar.com/c/omniscan-450/application-programming-interface.
Expected Behavior
When setting speed of sound on OMNISCAN450, the library should use message ID 116.
Current Behavior
Library sends/expects ID 1002, causing the device to ignore the message (returns a 'not-acknowledged'/NACK response)
Steps to Reproduce
- Create OMNISCAN450 instance
- Call set_speed_of_sound() method
- Device does not update speed of sound (ignores command)
Environment
- Device: Cerulean Omniscan450 FS
- Library: bluerobotics-ping (latest)
- OS: Ubuntu (WSL 2)
Severity
High - breaks core feature (speed of sound configuration)
Proposed Solution
Change omniscan450.json set_speed_of_sound ID from 1002 to 116.
Two PR's have been created;
- https://github.com/bluerobotics/ping-protocol/pull/221: fixes the core issue in the omniscan450.json file in the ping-protocol repository.
- https://github.com/bluerobotics/ping-python/pull/170: ensures the fix in the ping-protocol repository will be propagated to the ping-python package.
- Lingua principale
- Python
- Stelle
- 60
- Fork
- 39
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di bluerobotics/ping-python
-
avoid using print() function Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
bluerobotics/ping-python#141 · 2 commenti · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
bluerobotics/ping-python#74 ·
-
set sound speed Aperta
bluerobotics/ping-python#169 · 2 commenti · 1 assegnatario ·
-
Code after return Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
bluerobotics/ping-python#167 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 32/100
bluerobotics/ping-python#159 ·
Tutte le issue di bluerobotics/ping-python
Issue simili
-
essnmx good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
-
[Feature] 奇物选择添加优先级 Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Giskard-AI/giskard-oss#2840 · 1 commento ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Apertaarea: repo bug perceived difficulty: 2
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
yeti-platform/yeti#1380 ·