Skip to content

ui: fix long maneuver toggle#37622

Merged
elkoled merged 2 commits intomasterfrom
fix-long-maneuver
Apr 23, 2026
Merged

ui: fix long maneuver toggle#37622
elkoled merged 2 commits intomasterfrom
fix-long-maneuver

Conversation

@elkoled
Copy link
Copy Markdown
Contributor

@elkoled elkoled commented Mar 9, 2026

Fixes #37623

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 9, 2026

UI Preview

mici: ⚠️ Videos differ! View Diff Report
big: ⚠️ Videos differ! View Diff Report

Comment on lines -125 to -127
if not long_man_enabled:
self._long_maneuver_toggle.action_item.set_state(False)
self._params.put_bool("LongitudinalManeuverMode", False)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is there anything that resets on second onroad transition?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

CLEAR_ON_OFFROAD_TRANSITION in params_keys.h

Confirmed it resets in Lexus on mici.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

This PR has had no activity for 24 days. It will be automatically closed in 7 days if there is no activity.

@github-actions github-actions Bot added the stale label Apr 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.

@github-actions github-actions Bot closed this Apr 12, 2026
@elkoled elkoled reopened this Apr 23, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Process replay diff report

Replays driving segments through this PR and compares the behavior to master.
Please review any changes carefully to ensure they are expected.

✅ 0 changed, 66 passed, 0 errors

@elkoled elkoled merged commit 49685fc into master Apr 23, 2026
12 checks passed
@elkoled elkoled deleted the fix-long-maneuver branch April 23, 2026 00:59
karelnagel pushed a commit to asiusai/openpilot that referenced this pull request Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI: Long maneuver report toggle deactivates on ONROAD transition

2 participants