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

Migrate TB3 to nav2 minimal turtlebot repository & add the TB4 bringup option for modern gazebo #4401

Merged
merged 47 commits into from
Jun 6, 2024

Conversation

SteveMacenski
Copy link
Member

@SteveMacenski SteveMacenski commented Jun 5, 2024

ahcorde and others added 30 commits June 16, 2023 17:03
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Suggestions for navigation2#3634
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
SteveMacenski and others added 7 commits May 28, 2024 14:57
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Copy link
Contributor

mergify bot commented Jun 5, 2024

This pull request is in conflict. Could you fix it @SteveMacenski?

Copy link
Contributor

mergify bot commented Jun 5, 2024

@SteveMacenski, your PR has failed to build. Please check CI outputs and resolve issues.
You may need to rebase or pull in main due to API changes (or your contribution genuinely fails).

Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Copy link
Contributor

mergify bot commented Jun 5, 2024

@SteveMacenski, your PR has failed to build. Please check CI outputs and resolve issues.
You may need to rebase or pull in main due to API changes (or your contribution genuinely fails).

@SteveMacenski SteveMacenski changed the title Migrate TB3 to nav2 minimal turtlebot repository & add the TB4 bringup option Migrate TB3 to nav2 minimal turtlebot repository & add the TB4 bringup option for modern gazebo Jun 5, 2024
@SteveMacenski SteveMacenski merged commit ebe60dc into main Jun 6, 2024
9 of 10 checks passed
@SteveMacenski SteveMacenski deleted the BR6 branch June 6, 2024 02:17
Marc-Morcos pushed a commit to Marc-Morcos/navigation2 that referenced this pull request Jul 4, 2024
…p option for modern gazebo (ros-navigation#4401)

* Initial support to the new Gazebo

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed build

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feeback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix copyright and urdf

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* min range cannot be zero

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Simplify files

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Try to reduce load

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed cast shadows

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use Gazebo plugins instead of gz_ros2_control

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* update dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed ros2_control and use ogre

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use param file to configure bridge

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use sdf model and change to ogre instead of ogre2 for sensors

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Support Garden and later, performance improvements

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Use xacro for world file (ros-navigation#2)

Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>

* Reviewer feedback

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Add comment explaining temp file

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Fix linter

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Update nav2_bringup/worlds/gz_world_only.sdf.xacro

Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

* port GZ Classic launch changes to modern GZ and remove GZ Classic launch file

* adding changes

* changes done for TB4

* tb3 sim prototype

* fix missing ref

* linting

* fix branch name

* remove assets

* removing refs to turtlebot3_gazebo

* update map

* add new depot map

* Update Nav2 side of multirobot launch files

* linting

---------

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Marc-Morcos pushed a commit to Marc-Morcos/navigation2 that referenced this pull request Jul 4, 2024
…p option for modern gazebo (ros-navigation#4401)

* Initial support to the new Gazebo

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed build

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feeback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix copyright and urdf

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* min range cannot be zero

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Simplify files

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Try to reduce load

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed cast shadows

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use Gazebo plugins instead of gz_ros2_control

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* update dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed ros2_control and use ogre

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use param file to configure bridge

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use sdf model and change to ogre instead of ogre2 for sensors

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Support Garden and later, performance improvements

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Use xacro for world file (ros-navigation#2)

Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>

* Reviewer feedback

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Add comment explaining temp file

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Fix linter

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Update nav2_bringup/worlds/gz_world_only.sdf.xacro

Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

* port GZ Classic launch changes to modern GZ and remove GZ Classic launch file

* adding changes

* changes done for TB4

* tb3 sim prototype

* fix missing ref

* linting

* fix branch name

* remove assets

* removing refs to turtlebot3_gazebo

* update map

* add new depot map

* Update Nav2 side of multirobot launch files

* linting

---------

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
savalena pushed a commit to savalena/navigation2 that referenced this pull request Jul 5, 2024
…p option for modern gazebo (ros-navigation#4401)

* Initial support to the new Gazebo

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed build

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feeback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix copyright and urdf

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* min range cannot be zero

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Simplify files

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Try to reduce load

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed cast shadows

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use Gazebo plugins instead of gz_ros2_control

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* update dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed ros2_control and use ogre

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use param file to configure bridge

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use sdf model and change to ogre instead of ogre2 for sensors

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Support Garden and later, performance improvements

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Use xacro for world file (ros-navigation#2)

Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>

* Reviewer feedback

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Add comment explaining temp file

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Fix linter

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Update nav2_bringup/worlds/gz_world_only.sdf.xacro

Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

* port GZ Classic launch changes to modern GZ and remove GZ Classic launch file

* adding changes

* changes done for TB4

* tb3 sim prototype

* fix missing ref

* linting

* fix branch name

* remove assets

* removing refs to turtlebot3_gazebo

* update map

* add new depot map

* Update Nav2 side of multirobot launch files

* linting

---------

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Manos-G pushed a commit to Manos-G/navigation2 that referenced this pull request Aug 1, 2024
…p option for modern gazebo (ros-navigation#4401)

* Initial support to the new Gazebo

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed build

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feeback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fixed physics tag

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix copyright and urdf

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* min range cannot be zero

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Simplify files

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Added feedback

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Fix

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Try to reduce load

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed cast shadows

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use Gazebo plugins instead of gz_ros2_control

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* update dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed dependency

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Removed ros2_control and use ogre

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use param file to configure bridge

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>

* Use sdf model and change to ogre instead of ogre2 for sensors

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Support Garden and later, performance improvements

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Use xacro for world file (ros-navigation#2)

Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>

* Reviewer feedback

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Add comment explaining temp file

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Fix linter

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>

* Update nav2_bringup/worlds/gz_world_only.sdf.xacro

Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

* port GZ Classic launch changes to modern GZ and remove GZ Classic launch file

* adding changes

* changes done for TB4

* tb3 sim prototype

* fix missing ref

* linting

* fix branch name

* remove assets

* removing refs to turtlebot3_gazebo

* update map

* add new depot map

* Update Nav2 side of multirobot launch files

* linting

---------

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisuzt@intrinsic.ai>
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