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

ROS 2 Jazzyへの対応 #13

Merged
merged 22 commits into from
Nov 28, 2024
Merged

ROS 2 Jazzyへの対応 #13

merged 22 commits into from
Nov 28, 2024

Conversation

KuraZuzu
Copy link
Contributor

@KuraZuzu KuraZuzu commented Nov 27, 2024

What does this implement/fix?

ROS 2 Jazzyに対応します。

Does this close any currently open issues?

しません

How has this been tested?

動作環境

  • Raspberry Pi Mouse V3
    • Raspberry Pi 4B (4GB)
    • Ubuntu Server 24.04 (6.8.0-1014-raspi)
    • ROS 2 Jazzy Base
    • LiDAR: urg
  • Remote PC
    • Ubuntu Desktop 24.04 (6.8.0-49-generic)
    • ROS 2 Jazzy Desktop
    • Gazebo Harmonic v8.6.0

動作手順

README.mdの手順に沿って動作を確認しました。

Any other comments?

  • collision_monitorの機能は、現時点では暫定的に実質無効化しています。

Checklists

[補足]コストマップは読み込めるがナビゲーションできない状態
…方法の記述を追加(しかし、デフォルトを指定されるようにコメントアウト)
[補足]"controller_server"では"True"にするとNavigationが開始されない。
[補足]シミュレータ上での動作のみを確認した
@KuraZuzu KuraZuzu added the Type: Feature New Feature label Nov 27, 2024
@KuraZuzu KuraZuzu requested a review from YusukeKato November 27, 2024 02:35
@KuraZuzu KuraZuzu self-assigned this Nov 27, 2024
Copy link
Contributor

@YusukeKato YusukeKato left a comment

Choose a reason for hiding this comment

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

レビューコメントの確認をお願いいたします。

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
raspimouse_navigation/launch/pc_navigation.launch.py Outdated Show resolved Hide resolved
Copy link
Contributor

@YusukeKato YusukeKato left a comment

Choose a reason for hiding this comment

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

追加でコメントします。

README.md Outdated Show resolved Hide resolved
@YusukeKato
Copy link
Contributor

実機でSLAMとNavigationの動作確認できました!
コメントの確認&修正をお願いいたします。

@KuraZuzu
Copy link
Contributor Author

KuraZuzu commented Nov 28, 2024

コメントに沿った修正をした上で、実機とシミュレーション上で動作確認を行いました。
実機での動作を確認できました。一方、raspimouse_simから実行するシミュレーション上でのナビゲーションの動作を確認できませんでしたが、以下のコマンドの通りにuse_sim_timeを指定すると動作を確認できました。

ros2 launch raspimouse_navigation pc_navigation.launch.py use_sim_time:=True map:=$HOME/MAP_NAME.yaml

シミュレーションでは、launchファイルのuse_sim_timeのデフォルトがTrueであることを期待した動作なので、raspimouse_slam_navigation_ros2でのuse_sim_timeのデフォルトをTrueに戻すか、raspimouse_sim側のREADMEを修正する必要があります。

@YusukeKato
Copy link
Contributor

LGTMです!
raspimouse_simのREADMEのコマンドのほうを修正お願いいたします。

@YusukeKato YusukeKato merged commit 0357d49 into main Nov 28, 2024
1 check passed
@YusukeKato YusukeKato deleted the feature/support-jazzy branch November 28, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants