Omniscan 450: Cannot change speed of sound. Wrong message id.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 25/100
Línea de trabajo
Comienza con src/definitions/omniscan450.json en el repositorio ping-protocol enlazado y revisa el PR #221 de ping-protocol; después, comprueba cómo el PR #170 de ping-python propaga el cambio. Se considera terminado cuando la operación set_speed_of_sound de OMNISCAN450 usa el ID de mensaje 116 y el dispositivo ya no devuelve un NACK.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 60
- Forks
- 39
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de bluerobotics/ping-python
-
avoid using print() function Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
bluerobotics/ping-python#141 · 2 comentarios · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
bluerobotics/ping-python#74 ·
-
set sound speed Abierto
bluerobotics/ping-python#169 · 2 comentarios · 1 asignado ·
-
Code after return Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
bluerobotics/ping-python#167 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 32/100
bluerobotics/ping-python#159 ·
Todos los issues de bluerobotics/ping-python
Issues similares
-
bug ci good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
documentation
Dificultad 2/5 Medio día Aptitud para principiantes 62/100
inmanta/inmanta-core#10835 ·
-
sponsored
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
opensubtitlescom: moviehash never sent when opensubtitles (.org) is not in the provider list Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Diaoul/subliminal#1382 ·