Skip to content

Fix: VisualizePose port typo in MPC pose tracking Objectives#608

Draft
griswaldbrooks wants to merge 1 commit intomainfrom
fix/mpc-visualizepose-port-typo
Draft

Fix: VisualizePose port typo in MPC pose tracking Objectives#608
griswaldbrooks wants to merge 1 commit intomainfrom
fix/mpc-visualizepose-port-typo

Conversation

@griswaldbrooks
Copy link
Copy Markdown

Summary

Two lab_sim Objectives have a transposed-word typo on a VisualizePose Behavior port: it reads {stamped_pose} but the upstream producer writes {pose_stamped}. The variable doesn't exist on the blackboard at tick time, so the Behavior fails immediately and takes the Objective with it.

This is a deterministic failure that fires every CI run on main, contributing to the broken integration-test-in-studio-container (humble) job.

Files affected:

  • src/lab_sim/objectives/mpc_pose_tracking_dynamic_point_cloud_avoidance_with_sphere_down_sample.xml:27
  • src/lab_sim/objectives/mpc_pose_tracking_static_point_cloud_avoidance_with_sphere_down_sample.xml:20

In each file the producer is the line directly above (ForEach … out="{pose_stamped}" in the dynamic case, CreatePoseStamped pose_stamped="{pose_stamped}" in the static case), so the fix is a one-token swap.

Failing log signature on main:

[error] VisualizePose Error: Failed to get required values from input data ports:
The input port 'pose' was set to '{stamped_pose}', which was not found.
Please verify that this is the correct blackboard variable name.

Test plan

  • CI integration-test-in-studio-container (humble) runs MPC Pose Tracking Dynamic Point Cloud Avoidance with Sphere Down Sample and MPC Pose Tracking Static Point Cloud Avoidance with Sphere Down Sample to SUCCESS (was failing on main).
  • No other Objective references {stamped_pose} (verified by grep — only these two files used the typo'd name).

🤖 Generated with Claude Code

The VisualizePose Action read from {stamped_pose}, but the upstream
producer writes {pose_stamped} — the two words are transposed. Both
dynamic and static MPC pose tracking with point cloud avoidance
Objectives fail at tick time with "input port 'pose' was set to
'{stamped_pose}', which was not found", causing deterministic CI
failures in lab_sim integration tests.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
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.

1 participant