Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken Wheel Button Logic #33758

Open
nworb-cire opened this issue Oct 9, 2024 · 2 comments
Open

Broken Wheel Button Logic #33758

nworb-cire opened this issue Oct 9, 2024 · 2 comments
Labels
bug car vehicle-specific
Milestone

Comments

@nworb-cire
Copy link
Contributor

Describe the bug

Sometime since the opendbc migration, GM's steering wheel button logic has become broken. The resume button is now functionally equivalent to the set button. By this I mean that it will always cause the speed to be set to max(65 if experimental else 25, vEgo) instead of the previous set point as is desired.

Which car does this affect?

Chevrolet Volt 2018

Provide a route where the issue occurs

806907cd33ef2647/0000001b--0c38edf676/0

openpilot version

596d8b1

Additional info

I have confirmed with a few other GM users that they experience this bug on master(-ci). You can see in the first segment of this route the broken behavior.

(Note: dirty git required to remove my car from dashcam mode)

@nworb-cire nworb-cire added bug car vehicle-specific labels Oct 9, 2024
@adeebshihadeh adeebshihadeh added this to the 0.9.8 milestone Oct 9, 2024
@jyoung8607
Copy link
Collaborator

jyoung8607 commented Oct 11, 2024

Not GM specific, just had this on my Golf.

@sshane sshane changed the title GM: Broken Wheel Button Logic Broken Wheel Button Logic Oct 17, 2024
@adeebshihadeh
Copy link
Contributor

@sshane seems to be this: #33693. It now waits for a loopback on the enabled, and there's no more buttonEvents.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug car vehicle-specific
Projects
None yet
Development

No branches or pull requests

3 participants