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

Sagittaドライバの修正 #115

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Conversation

seki-hiro
Copy link
Member

@seki-hiro seki-hiro commented Jul 6, 2023

概要

Sagittaドライバの修正

Issue

#21

詳細

  • bootしたかどうかのフラグ管理方法の見直し
    bootしていない状態でSagittaとテレコマ通信するモードも存在する。パラメータ変更等のアサーションにbootフラグは使わない。
  • キャスト不具合修正

検証結果

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

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

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

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

bootフラグが立っていなくても、パラメータset/readコマンドが弾かれないことを確認。

camera.offset_pixに負の値を入れられることを確認。
image

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

  • 図や表で記述する

影響範囲

XX系の動作がガラッと変わる,とか.

補足

何かあれば

注意

  • 6U AOCS team Projects への紐付けを行うこと
  • Assignees を自分に設定すること
  • Reviewers を設定すること
  • priority ラベルやmajor/minor/patch updateラベルを付けること

@seki-hiro seki-hiro self-assigned this Jul 6, 2023
@seki-hiro seki-hiro changed the title WIP: Sagittaドライバの修正 Sagittaドライバの修正 Jul 10, 2023
@seki-hiro
Copy link
Member Author

@200km 検証が完了してWIPを外したので、reviewお願いいたします。

もともと実機STTでの検証を想定していましたが、EM到着が金曜日以降になるとのことで、先にマージして他の修正も加えたいと考えています。
検証結果に記載の通り、必要な検証はSILSでできていると思います。

@seki-hiro seki-hiro requested a review from 200km July 10, 2023 13:30
Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

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

問題ないです。

@seki-hiro seki-hiro merged commit 04dd091 into develop Jul 11, 2023
@seki-hiro seki-hiro deleted the feature/update_sagitta_driver branch July 11, 2023 02:06
@seki-hiro seki-hiro mentioned this pull request Aug 14, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants