search

ut-issl / s2e-core

Spacecraft Simulation Environment Core codes
MIT License
46 stars 18 forks source link

Add Relative Velocity Direction Pointing #617

Closed Hiro-0110 closed 5 months ago

Hiro-0110 commented 6 months ago

Related issues

N/A

Description

Add relative velocity direction from ground to spacecraft pointing mode to controlled attitude list

Test results

N/A

Impact

Describe the scope of influence of the changes, e.g., The behavior of feature ** changes.

Supplementary information

Provide any supplementary information.

Hiro-0110 commented 6 months ago

修正提案ありがとうございます。動作確認結果も載せてもらえると嬉しいです。

動作確認の流れがあまり見えていないのですが、controlled attitudeのモードを相対速度方向に変更してs2eが動けば良いということでしょうか

200km commented 6 months ago

実装が正しいと確認するために何かしら動作確認をするのかなと思います。その結果を共有してもらえると嬉しいです。動作確認方法はどんな実装かによって変わりますが、まずはコード書いた人が動作確認結果を示し、レビュー時に追加で検証が必要と判断されたら、どんな試験を追加ですべきかレビュワーと相談して決めていくのが良いと思います。

今回の場合、提案してくれているようにモードを変更して思い通りの動作になっているかを図などで示してくれれば良いかなと思います。

Hiro-0110 commented 5 months ago

検証として、今回新たに実装した相対速度方向と、元々の衛星の速度方向の差分をプロットしました。

z成分が0になっていて、地球の自転に応じて, x, y成分が一周していることがわかりました。