Skip to content

fix: stamp planner outputs#17

Open
lblommesteyn wants to merge 1 commit into
WE-Autopilot:mainfrom
lblommesteyn:fix/stamp-planner-outputs
Open

fix: stamp planner outputs#17
lblommesteyn wants to merge 1 commit into
WE-Autopilot:mainfrom
lblommesteyn:fix/stamp-planner-outputs

Conversation

@lblommesteyn

Copy link
Copy Markdown
Contributor

Problem
Planner publishes TargetPathStamped and SpeedProfileStamped messages with default/zero headers. During smoke testing, target_path echoed with sec: 0, nanosec: 0.

Downstream consumers and debugging tools should be able to trust the planner output timestamps/frame ids.

Changes

  • Stamp TargetPathStamped before publishing.
  • Stamp SpeedProfileStamped before publishing.
  • Use the current lane frame_id for both output frame ids.

Test plan

  • colcon build --symlink-install --packages-select ap1_planning --cmake-args -DCMAKE_BUILD_TYPE=Release
  • git diff --check

@lblommesteyn lblommesteyn requested a review from a team as a code owner June 5, 2026 03:16
@lblommesteyn

Copy link
Copy Markdown
Contributor Author

Verified locally in the software smoke-test stack: /ap1/planning/target_path now publishes with a nonzero header stamp and base_link frame_id.

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