Skip to content

#KL26-90 IMU直進クラスの作成#19

Merged
HaruArima08 merged 8 commits into
mainfrom
ticket-KL26-90
May 23, 2026
Merged

#KL26-90 IMU直進クラスの作成#19
HaruArima08 merged 8 commits into
mainfrom
ticket-KL26-90

Conversation

@migaku2645

@migaku2645 migaku2645 commented May 19, 2026

Copy link
Copy Markdown
Contributor

チェックリスト

  • clang-format している
  • コーディング規約に準じている
  • チケットの完了条件を満たしている

変更点

  • IMUセンサを使用した直進のための変更
  • Straightクラスの引数に角度制御用PIDゲイン、IMU使用フラグ追加
  • メンバ変数に目標角度=0を追加
  • 既存のテストにIMU有りの場合を追加

動作テスト

https://www.notion.so/uom-katlab/IMU-360dd5b1cc18800fb69ce3d372036bf6?source=copy_link

実験方法

実験結果

添付資料

角度正規化クラス追加しました。レビューお願いします!

@HaruArima08 HaruArima08 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

見ました。
動作確認もできてるし特に問題ないと思います。
細かい修正とコメント書いてます。
よろしくお願いします。

Comment thread modules/motions/Straight.h Outdated
Comment thread tests/StraightTest.cpp Outdated
Comment thread tests/StraightTest.cpp
Comment thread modules/motions/Straight.cpp Outdated
@migaku2645 migaku2645 changed the title Ticket kl26 90 #KL26-90 IMU直進クラスの作成 May 20, 2026

@nishijima515 nishijima515 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

特に指摘するところはなかったので、先にapprove出しときます。LGTM!
LGTM

@takuchi17 takuchi17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ほぼ問題ないと思います。

角度の正規化についてだけ、imuタスクと相談して同じ関数を使うようにしたい。imuタスクのコメント参考。

Comment thread modules/motions/Straight.h Outdated
Comment thread modules/motions/Straight.cpp Outdated
@HaruArima08

Copy link
Copy Markdown
Collaborator

calculatorsに角度の正規化の作成は結局どっちでやることになってる?
直進の方が回頭より先にマージできそうだから、もうこっちで作って、回頭側は最新のmainマージして対応してもらった方が良いと思うんだけど。

@migaku2645
migaku2645 requested a review from a team May 23, 2026 04:20

@HaruArima08 HaruArima08 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

対応ありがとうございます!
お疲れ様!

LGTM

@takuchi17 takuchi17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@HaruArima08
HaruArima08 merged commit 228c0af into main May 23, 2026
10 checks passed
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.

4 participants