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

gnssnavの処理面の問題 #117

Open
wants to merge 26 commits into
base: deploy
Choose a base branch
from
Open

gnssnavの処理面の問題 #117

wants to merge 26 commits into from

Conversation

kyo0221
Copy link
Contributor

@kyo0221 kyo0221 commented Sep 13, 2024

#111 で挙げられた処理面の問題を解決する.

  • 変数の0リセットが出来てない <-ある程度直したが
  • 上に付随してros msgの構造体を使用して可読性を上げた方が良い
  • メンバ変数をなるべく使わないようにと,定数と変数の区別をつける <-デバッグがしんどくなる,欲を言えばコンパイル時に処理を終わらせときたい.
  • 快くshared_ptrで来ているのに中身を値移ししてのは,処理面でもrosの仕様上も良くない.point_ = msg->poses;<-これとか
  • vectornav/pose吐いたまま起動できない.(運が良いと起動する) 重要
  • スタートの実装が欲しい <-controllerからもできるようにしたい
  • プロセスを終了せずにリスタートしたい 重要
  • たまにゆっくりになってしまう <-purepersuitの先読みの問題
  • 角加速度のゲインチューニングができない

HarukiOgawa1 and others added 25 commits April 29, 2024 15:23
@kyo0221
Copy link
Contributor Author

kyo0221 commented Sep 13, 2024

プロセスを終了せずにリスタートする機能を追加しました.

ros2 topic pub --once /nav_start std_msgs/msg/Empty

を実行することで目標地点が初期地点に戻ります.

以下が挙動を確認した動画です.
https://youtu.be/tRMisAptvCs

@kyo0221
Copy link
Contributor Author

kyo0221 commented Sep 13, 2024

vectornav/poseを付けた状態では起動できない問題を修正しました.

path_publisherを受け取る前にvectornav/poseを受け取った時にエラーが発生しているようだったので, pathの中身が空であればvectornavのcallback関数をreturnで戻す処理を追加することで対処しました.

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.

3 participants