Skip to content

fix(planning): latch max_vel with transient_local QoS - #3634

Merged
shmpwk merged 1 commit into
feat/v0.64/e2e_scenario_passfrom
fix/v0.64/e2e_scenario_pass/external_velocity_limit_initial_max_vel
Oct 21, 2026
Merged

shmpwk merged 1 commit into
feat/v0.64/e2e_scenario_passfrom
fix/v0.64/e2e_scenario_pass/external_velocity_limit_initial_max_vel

Conversation

@kotaro-hihara

@kotaro-hihara kotaro-hihara commented Oct 20, 2026 •

Copy link
Copy Markdown

backport
tier4/pilot_auto_planning#14

An empty clear was published as max_velocity 0 because hardest_limit_ stayed at its default. Initialize it from max_vel so departure is not blocked.

slack

Publish the configured max_vel when no sender has set a limit, using transient_local so subscribers that start later still receive it.

Co-authored-by: Cursor <cursoragent@cursor.com>
@mergify

mergify Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@sonarqubecloud

Copy link
Copy Markdown

@shmpwk
shmpwk merged commit 8f3bf9d into feat/v0.64/e2e_scenario_pass Oct 21, 2026
4 checks passed
@shmpwk
shmpwk deleted the fix/v0.64/e2e_scenario_pass/external_velocity_limit_initial_max_vel branch October 21, 2026 23:44
shmpwk pushed a commit to tier4/pilot_auto_planning that referenced this pull request Nov 5, 2026
…_universe#3634)

Publish the configured max_vel when no sender has set a limit, using transient_local so subscribers that start later still receive it.

Co-authored-by: Cursor <cursoragent@cursor.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