-
Notifications
You must be signed in to change notification settings - Fork 63
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
Update to ROS 2 Jazzy and make TurtleBot naming generic #52
Conversation
f71e63e
to
0309c96
Compare
6d43f18
to
d4b795b
Compare
d4b795b
to
e5972a1
Compare
@elsayedelsheikh This was working until I renamed all the I'm basically getting a disconnected TF tree in the Nav2 setup, but I can't figure out what is misconfigured. EDIT: ... never mind, I did a clean rebuild and it works now. Could you review regardless? |
So far so good, I think turtlebot 4 is spawned with
And if the default is turtlebot 3, this one needs to be updated as well turtlebot4_bridge.yaml -> turtlebot3_bridge.yaml
EDIT: Maybe this isn't the best practice (
|
Thanks @elsayedelsheikh -- good catch! I just made those fixes and was able to run the TB3 and TB4 examples. Might merge this, and then we can make other "nice to have" fixes as we find time. |
This PR upgrades to ROS 2 Jazzy and replaces most mentions of TurtleBot3 /
tb3
with generic TurtleBot /tb
We needed #51 merged in first, since Gazebo classic is not in this version of ROS 2.