Skip to content

[Backport release/0.0.14] fix(g1): route planner and teleop velocities to the sim connection - #4175

Open
github-actions[bot] wants to merge 1 commit into
release/0.0.14from
backport/4150-to-release/0.0.14
Open

github-actions[bot] wants to merge 1 commit into
release/0.0.14from
backport/4150-to-release/0.0.14

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Description

Backport of #4150 to release/0.0.14.

@greptile-apps

greptile-apps Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

Safe to merge: the added blueprint component completes the expected velocity-command routing to the simulated G1 connection.

What we checked:

  • Authored the executable g1-movement-routing-check.py to validate movement-routing behavior. T-Rex
  • Ran the before revision of the script from the repository, and it exited with code 1. T-Rex
  • Ran the after revision of the script from the repository, and it exited with code 0. T-Rex
  • The final output decisively reported nav_cmd_vel, tele_cmd_vel, and cmd_vel states, confirming the observed routing behavior after the revision. T-Rex

Summary

This change adds MovementManager to the basic G1 simulation blueprint so planner and teleoperation velocity commands share the route to the simulation connection. The checked command-routing composition is complete and no issues were found.

Reviews (1) · Last reviewed commit: "fix(g1): route planner and teleop veloci..."

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