Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update HK #225

Merged
merged 3 commits into from
Sep 27, 2023
Merged

Update HK #225

merged 3 commits into from
Sep 27, 2023

Conversation

200km
Copy link
Member

@200km 200km commented Sep 25, 2023

Issue

#196

詳細

運用経験をもとにHKの整理を行なった。主な変更は次のとおり。

  • HK_GEN
    • EL/EH関連テレメの整理
      • LOWレベルテレメの追加
      • EH発生時刻、ルールIDの追加
    • OC閾値は基本固定値なのでHKからは削除(設定値を確認したい場合は別TLMで)
  • HK_COMPO
    • SS測定角度の削除
    • SS数の増加
    • STT観測星数の追加
  • HK_ALGO
    • Error情報の削除
      • Error情報は推定値と目標値から地上局で計算できる
    • RWアンローディング閾値の削除(設定値を確認したい場合は別TLMで)
    • 慣性座標系での太陽方向、磁場ベクトルの追加
    • Quaternion内挿モードのIDX追加

検証結果

ビルドチェック (どちらもチェック)

  • SILSでのビルドチェックに通った(CIで確認)
  • vMicroでのビルドチェックに通った

動作確認チェック (いずれかをチェック)

  • SILSでアルゴリズムが想定通りに動いた
  • 実機でアルゴリズムが想定通りに動いた
  • (テレコマ試験の場合)コマンドファイルを使った試験をパスした

試験結果詳細記述場所 or 詳細ログ保存場所へのリンク

SILS結果
HK_GEN
image

HK_COMPO
image

HK_ALGO
image

補足

議論資料(privateファイル)

@200km 200km requested review from sksat and a team as code owners September 25, 2023 14:52
@200km 200km requested review from suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team September 25, 2023 14:52
@200km 200km self-assigned this Sep 25, 2023
@200km 200km added ✈️ priority::medium priority medium 🐳 major update Major update labels Sep 25, 2023
@200km 200km added this to the v8.0.0 Major update milestone Sep 25, 2023
@200km 200km linked an issue Sep 25, 2023 that may be closed by this pull request
Copy link
Member

@conjikidow conjikidow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。
こちら(private)にあるように,QUATERNION.ERROR_B2T_XなどはWINGSで計算できるようになってから削除するとの方針だったと記憶しているのですが,その前に消すことになったんでしたっけ?

@200km
Copy link
Member Author

200km commented Sep 26, 2023

プロジェクト固有でこのMajor updateをいつ反映するかは好きに選べるので、WINGS整備後でも良いと思います。
#199 もありますので、どうしても必要ならprivateプロジェクト内ではerrorを残すように修正しても良いとは思います。

@conjikidow
Copy link
Member

プロジェクト固有でこのMajor updateをいつ反映するかは好きに選べる

というのは確かにそうですが,c2aとしてサポートしているWINGSにその機能がない以上,このリポジトリでもまだ消さないほうが良いのかなと思いますがいかがでしょうか。

@200km
Copy link
Member Author

200km commented Sep 26, 2023

いろんな論点がある気がしますが、下記のような点から今の時点での修正で良いと思います。

  • Errorテレメがないとc2a-aobcのシステム全体が動かないというわけではない
    • エラーなどが出てWINGSと通信できなくなるわけではない
    • Errorテレメはあると便利という程度のテレメトリで、他に消したものと同様HKとしては優先度が低いので消している
    • WINGS上でリアルタイムに計算・表示できなくても、後解析などでは問題なく計算できるものである
  • c2aとwingsは連携しているが完全に歩調を合わせないといけないわけではない
    • どちらかが先行して機能を追加するなどはこれまでもやってきた(Cmd prefixの件など)
    • 逆にどちらかが先に変えないと、一方がそれに合わせるモチベーションが湧かない
  • 必要なら(他の消したテレメトリと同様)HK以外で下ろせば良い
    • RWアンローディング閾値などもそうだが、今回消したテレメを整理して別パケットのテレメに加える予定(privateスプレッドシートに記載している通り)
    • もしくは既に書いたようにプロジェクト毎にTLM設定できるのでそこでチューニングすることができる

重要度という観点でも低いですし、どうしても使いたい人には代替案が用意されるような状況のものですので問題ないと思います。

Base automatically changed from feature/fix-tlm-db-small to develop September 26, 2023 10:59
Copy link
Member

@conjikidow conjikidow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

そうですね,わかりました。approveします。

@200km 200km merged commit 5487b7c into develop Sep 27, 2023
13 of 15 checks passed
@200km 200km deleted the feature/update-hk branch September 27, 2023 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HKの更新
2 participants