Add speed of sound estimation from water properties

オープン
#125 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
38/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
python
領域
backend, data

調査の方向性

まず既存のPingDevice実装と提案されているwater.pyモジュールを確認し、その後、bluerobotics/ping-viewer/issues/1003のサンプルv_soundコードを、リンクされている塩分推定のガイダンスと比較します。完了条件は、音速と塩分の推定を提供し、PingDevice.estimate_v_sound(depth, temperature, salinity)を公開し、指定されたデフォルト値を初期化することです。NASAデータへのアクセスは引き続きTODOとします。

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

説明

enhancement good first issue

Would help with #122, and would also better enable data processing capabilities.

Should probably be implemented as a water.py module, with a helper for estimating salinity as per here. Ideally there would be some kind of API access to NASA's global sea surface salinity data (queried by GPS location?) or something, but that is probably best left as a note/TODO for now.

Example v_sound code at bluerobotics/ping-viewer/issues/1003

Would be useful to include an estimate_v_sound(self, depth=None, temperature=None, salinity=None) method in PingDevice that uses water.estimate_sound_speed(depth, temperature, salinity), which can be updated over time to use the most up to date implemented method. PingDevice can have its initialisation updated to include initial depth=0, temperature=25, salinity=water.estimate_salinity_from_type("seawater") values.

主要言語
Python
スター
60
フォーク
39
30日以内にマージされた PR はありません

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

bluerobotics/ping-python のほかの issue

bluerobotics/ping-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

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

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