Skip to content

Add turtlebot3 example #193

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 13 commits into from
Nov 9, 2025
Merged

Add turtlebot3 example #193

merged 13 commits into from
Nov 9, 2025

Conversation

@Yutarop
Copy link

@Yutarop Yutarop commented Nov 6, 2025

Hi

I’ve known about this ROS MCP server for a few months (from a Reddit post) but hadn’t had a chance to try it out until recently. i finally tested it using a TurtleBot3, but since i couldn’t find an example for TurtleBot3, I created one myself.

By following this tutorial, i could control a real TurtleBot3 in the real world. Please let me know if there are any parts of the tutorial that could be improved or are missing.

Thanks!

@stex2005
Copy link
Collaborator

stex2005 commented Nov 7, 2025

hanks, @Yutarop — this is a great contribution!
The example is clear and will be especially useful for users running TurtleBot3 setups.

A few suggestions before merging:

  • Could you briefly mention which ROS 2 distro and firmware configuration you used (e.g., ros2 humble, burger/waffle)?
  • Consider linking directly to your example launch file or tutorial section in the README for easier discovery.
  • The GIF demo is a nice touch — maybe compress or resize it slightly to keep the repo size manageable.

Once those are updated, I think it’s ready to merge. Thanks again for putting this together!

@stex2005 stex2005 requested a review from rjohn-v November 7, 2025 16:28
rjohn-v
rjohn-v approved these changes Nov 7, 2025
View reviewed changes
Copy link
Contributor

@rjohn-v rjohn-v left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thank you! This example looks clean and easy to follow. I'm happy to have it in our repo.
Approved from my end. (I agree with Stefano that it is ok to link to your repo for people to see more details and to discover you as the contributor)

@Yutarop , I just looked up your other posts. Great to see your projects trying to do more with ROS and MCP.
If you're interested in collaborating more deeply with us at robotmcp, lets talk! https://www.linkedin.com/in/rohitjohnv/

@Yutarop
Copy link
Author

Yutarop commented Nov 8, 2025

Thank you for your review! @stex2005 @rjohn-v

i made a few changes below;

  • added the specifications of the robot used in the demo to the README.
  • added a launch file to run TurtleBot3 and the ROS MCP server.
  • compressed the GIF file from 7.19 MB to 2.7 MB.

And, thank you for your invitation. I'd love to collaborate!

@stex2005
Copy link
Collaborator

stex2005 commented Nov 8, 2025

Final fix before merge. Can you please run the linter with ruff format --check and commit? Otherwise I cannot commit

@Yutarop
Copy link
Author

Yutarop commented Nov 9, 2025 •
edited
Loading

@stex2005
I applied formatting to the launch file. Please merge if it looks good.

@stex2005
Copy link
Collaborator

stex2005 commented Nov 9, 2025

Ruff format too please

@stex2005 stex2005 merged commit ddcfff1 into robotmcp:develop Nov 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rjohn-v rjohn-v rjohn-v approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Yutarop @stex2005 @rjohn-v