-
Notifications
You must be signed in to change notification settings - Fork 22
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
base: main
Are you sure you want to change the base?
Conversation
@shmpwk
8月末(8/28?)に実験実験できたらして、問題なさそうでしたらfalseにしたいなと思います。 |
バージョン管理はしていません。 |
There was a problem hiding this 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に委ねます。
マイナーリリースではハッシュ変わらないですか? |
700aec7
to
35687b3
Compare
はい、変わらないので大丈夫だと思います! |
…ctly Signed-off-by: kosuke55 <[email protected]>
35687b3
to
aabceea
Compare
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
254bf47
to
1816618
Compare
Signed-off-by: kosuke55 <[email protected]>
タイヤ角度 -> ハンドル角度への変換処理をraw_vehicle_cmd_converterに移動しようとしています。
autowarefoundation/autoware.universe#8504
これは
- feat(simple_planning_simulator): add VGR sim model autowarefoundation/autoware.universe#8415
という背景です。
と、デフォルトではこれまで通りpacmod_interface側で変換を行うようにしています。(機能が安定したらfalseにしたいです)
またactuation_statusのsteer_statusとしては、タイヤ角度ではなくて、ハンドル角度を返すように変更しています。
これは raw_vehicle_cmd_converter でタイヤ角度に変換することを期待しているためです。現状このトピックは使われていないはずで、変更は動作に影響ないと思います。