Skip to content

Simulation tutorial patches#45

Open
kjy5 wants to merge 3 commits intohello-robot:masterfrom
kjy5:simulation-fixes
Open

Simulation tutorial patches#45
kjy5 wants to merge 3 commits intohello-robot:masterfrom
kjy5:simulation-fixes

Conversation

@kjy5
Copy link
Copy Markdown

@kjy5 kjy5 commented Apr 15, 2026

In addition to #42, there are a couple more patches to fix incorrect filenames and potentially missing environment variables.

My changes do not duplicate/include #42.

Fixes:

  • Recommend running source ~/.bashrc after creating the workspace to ensure environment variables are added.
  • Patch the controller calibration file to the factory default one (since a calibration file will not be created in the demo).
  • Copy and rename certificates to the expected names by the web interface launch scripts.

Copilot AI review requested due to automatic review settings April 15, 2026 21:20
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the ROS 2 Stretch simulation tutorial to improve “out-of-the-box” setup for the web teleop demo by clarifying environment setup steps and adding patches/workarounds for expected files.

Changes:

  • Instructs users to source ~/.bashrc after adding the workspace setup line to .bashrc.
  • Updates the web interface section to explicitly use the uncalibrated URDF and adds a step to patch controller calibration copying.
  • Adds instructions to copy/rename certificate files to the names expected by the web teleop launch scripts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ros2/stretch_simulation.md Outdated
Comment thread ros2/stretch_simulation.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants