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

Add NaN assertion #323

Merged
merged 6 commits into from
May 29, 2024
Merged

Add NaN assertion #323

merged 6 commits into from
May 29, 2024

Conversation

200km
Copy link
Member

@200km 200km commented May 28, 2024

Issue

現状これを入れないと問題が出るというわけではないが、過去のバグを含め包括的にこの手の現象に対応するため。

詳細

数値計算上のNaNを回避するためにAssertionを入れた。特に過去のデータを引きずりエラーが継続しやすい次の箇所に入れた

  • PID制御
  • フィルター系

検証結果

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

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

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

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

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

  • 現時点で動かしてこのAssertionに引っ掛かることはなく、修正前後で制御結果などは変わりえない

補足

NA

@200km 200km added 🚀 priority::high priority high 🐬 minor update Minor update labels May 28, 2024
@200km 200km self-assigned this May 28, 2024
@200km 200km requested review from sksat and a team as code owners May 28, 2024 23:16
@200km 200km requested review from seki-hiro, suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team May 28, 2024 23:16
@200km
Copy link
Member Author

200km commented May 29, 2024

ELも追加しました。Spikeの方はエラーを返しており、そのエラーで上位アプリ側でELがかかるようになっているので、入れていません。

@conjikidow conjikidow added this to the v10.1 Minor update milestone May 29, 2024
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.

ありがとうございます。

@200km 200km merged commit 03d653f into develop May 29, 2024
9 of 11 checks passed
@200km 200km deleted the feature/add-assertion-nan branch May 29, 2024 01:39
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.

3 participants