ut-issl / s2e-core

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

[hotfix] fix force_generator #571

Closed TomokiMochizuki closed 9 months ago

TomokiMochizuki commented 9 months ago

Related issues

570

Description

Miscalculation of the force.

Test results

Before

image

After

image

Impact

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

Supplementary information

Provide any supplementary information.

200km commented 9 months ago

テスト結果を入れてもらえると助かります。 component内のコメントを外せば出力が出ると思いますし、python plotですぐ図示できます。

https://github.com/ut-issl/s2e-core/blob/a0c4b0627d4ab49b87da4d89bbabca166a0b354b/src/simulation_sample/spacecraft/sample_components.cpp#L147

https://github.com/ut-issl/s2e-core/blob/develop/scripts/Plot/plot_force_generator.py

TomokiMochizuki commented 9 months ago

テスト結果を入れてもらえると助かります。 component内のコメントを外せば出力が出ると思いますし、python plotですぐ図示できます。

https://github.com/ut-issl/s2e-core/blob/a0c4b0627d4ab49b87da4d89bbabca166a0b354b/src/simulation_sample/spacecraft/sample_components.cpp#L147

https://github.com/ut-issl/s2e-core/blob/develop/scripts/Plot/plot_force_generator.py

テスト結果を表示しました

200km commented 9 months ago

572 でTorqueGeneratorの出力を図示して分散値などをだしているように、ForceGeneratorの出力を図示した方が直接的な試験になるとおもいますがどうでしょう?

200km commented 9 months ago

plot_force_generatorの使い方が分からないなどあるのであれば、こちらで試験することも可能です。

TomokiMochizuki commented 9 months ago

コードは問題無いと思うのでApproveします。マージしてくれて良いですが、将来のユーザーのためにテスト結果の件については対応をお願いします。

テスト結果対応しましたのでマージします