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

パラメータの微修正 #56

Merged
merged 1 commit into from
Aug 11, 2023
Merged

パラメータの微修正 #56

merged 1 commit into from
Aug 11, 2023

Conversation

200km
Copy link
Member

@200km 200km commented Aug 9, 2023

Issue

NA

詳細

C2Aの下記PRとパラメータを合わせるための修正

ut-issl/c2a-aobc#163

検証結果

NA

補足

NA

注意

  • 必ずReviewers を設定すること
    • AOCSメンテナメンバー
  • Assignees を自分自身に割り当てること
  • Projectsとして6U AOCS team (private)を設定する
    • StatusWaiting Reviewに設定する
  • 必ずpriority ラベルを付けること
    • high: 試験などの関係ですぐさまレビューしてほしい
    • middle: 通常これ
    • low: ゆっくりで良いもの
  • 必ずupdateラベルをつけること
    • major: 後方互換性なしのI/F変更
    • minor: 後方互換性ありのI/F変更
    • patch: I/F変更なし
  • テンプレート記述は残さず、削除したりNAと書き換えたりする

@200km 200km added ✈️ priority::medium priority medium 🐟 patch Patch Update labels Aug 9, 2023
@200km 200km self-assigned this Aug 9, 2023
@200km 200km requested review from sksat and a team as code owners August 9, 2023 21:31
@200km 200km requested review from seki-hiro, suzuki-toshihir0 and conjikidow and removed request for a team August 9, 2023 21:31
@200km 200km added this to the 開発仮目標1 milestone Aug 9, 2023
@200km 200km added 🚀 priority::high priority high and removed ✈️ priority::medium priority medium labels Aug 10, 2023
Comment on lines +25 to +27
constant_bias_c_rad_s(0) = -0.0180004966
constant_bias_c_rad_s(1) = -0.00919875782
constant_bias_c_rad_s(2) = 0.0208997503
Copy link
Member

Choose a reason for hiding this comment

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

[Q] これは運用での推定結果の一つでしょうか?元々の値がメーカーの示すスペックであれば、そちらを使い続けた方がいいのかなと思いました。

Copy link
Member Author

Choose a reason for hiding this comment

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

元々の値がメーカーの示すスペックはなく、実機で測定する必要があります。また、個体差が結構大きいです。
この値自体の出所はわかりませんが、C2A側のデフォルト値に合わせています。

Copy link
Member

Choose a reason for hiding this comment

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

C2A側と合っていなかったということですね。わかりました。

@200km 200km requested a review from seki-hiro August 11, 2023 05:40
@200km 200km merged commit 9a8daa1 into main Aug 11, 2023
@200km 200km deleted the hotfix/fix-parameter branch August 11, 2023 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐟 patch Patch Update 🚀 priority::high priority high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants