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

feat(pacmod_interface): add option to receive steering wheel command directly #85

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kosuke55
Copy link

@kosuke55 kosuke55 commented Aug 16, 2024

タイヤ角度 -> ハンドル角度への変換処理をraw_vehicle_cmd_converterに移動しようとしています。
autowarefoundation/autoware.universe#8504

これは

  • そもそもcontrol cmd(タイヤ角度)をpacmod_interafaceで変換しているが責任範囲に違和感がある
  • vehicle interfaceごとに分かれた実装を共通化したい。

という背景です。

  /* parameter for steering conversion */
  enable_pub_steer_ = declare_parameter("enable_pub_steer", true);
  convert_steer_command_ = declare_parameter("convert_steer_command", true);

と、デフォルトではこれまで通りpacmod_interface側で変換を行うようにしています。(機能が安定したらfalseにしたいです)

またactuation_statusのsteer_statusとしては、タイヤ角度ではなくて、ハンドル角度を返すように変更しています。
これは raw_vehicle_cmd_converter でタイヤ角度に変換することを期待しているためです。現状このトピックは使われていないはずで、変更は動作に影響ないと思います。

actuation_status.status.steer_status = current_steer_wheel;

@kosuke55 kosuke55 requested a review from shmpwk August 16, 2024 09:41
@kosuke55 kosuke55 changed the title feat(pacmod_interface): Additional option to receive wheel angle directly feat(pacmod_interface): add option to receive wheel angle directly Aug 16, 2024
@kosuke55 kosuke55 changed the title feat(pacmod_interface): add option to receive wheel angle directly feat(pacmod_interface): add option to receive steering wheel command directly Aug 16, 2024
@kosuke55
Copy link
Author

@shmpwk
コードの変更内容的には良さそうか確認頂けないでしょうか? 🙏

(機能が安定したらfalseにしたいです)

8月末(8/28?)に実験実験できたらして、問題なさそうでしたらfalseにしたいなと思います。
raw_vehicle_cmd_converter側の変更も取り込まないと不整合がおきてしまうのですが、pacmod_interfaceのバージョン管理はどのようにしているのでしょうか?(常に最新の用でしたらuniverse側に合わせてtag切るとか必要かと思います

@shmpwk
Copy link
Contributor

shmpwk commented Aug 16, 2024

@kosuke55

pacmod_interfaceのバージョン管理はどのようにしているのでしょうか?(常に最新の用でしたらuniverse側に合わせてtag切るとか必要かと思います

バージョン管理はしていません。
リリース評価開始時にハッシュを固定して参照しています。
個別の実験であれば、実験用のブランチを作って管理していただくのが良いかと思います。

Copy link
Contributor

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

コードはLGTMです!設計部分はraw vehicle converterのcode ownwerに委ねます。

@kosuke55
Copy link
Author

リリース評価開始時にハッシュを固定して参照しています。

マイナーリリースではハッシュ変わらないですか?
universeに変更入らず、pacmod_interfaceだけバージョン上がってしまうことがないのであれば、問題なさそうです!

@kosuke55 kosuke55 force-pushed the feat/vgr_in_converter branch 2 times, most recently from 700aec7 to 35687b3 Compare August 16, 2024 10:36
@shmpwk
Copy link
Contributor

shmpwk commented Aug 17, 2024

@kosuke55

マイナーリリースではハッシュ変わらないですか?

はい、変わらないので大丈夫だと思います!

Signed-off-by: kosuke55 <[email protected]>
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.

2 participants