bluerobotics / bluerobotics/ping-python

Omniscan 450: Cannot change speed of sound. Wrong message id.

オープン
#171 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
60
フォーク
39
PR マージ指標
30日以内にマージされた PR はありません

説明

_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](https://github.com/bluerobotics/ping-protocol/issues/222) 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
1. Create OMNISCAN450 instance
2. Call set_speed_of_sound() method
3. 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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

リンクされている ping-protocol リポジトリの src/definitions/omniscan450.json から始めて、ping-protocol PR #221 を確認し、その後 ping-python PR #170 が変更をどのように反映しているかを確認してください。完了条件は、OMNISCAN450 の set_speed_of_sound operation が message ID 116 を使用し、デバイスが NACK を返さなくなることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
api
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。