openpilot
openpilot copied to clipboard
VW: update LKA HUD to be more logically accurate
Description
Original behavior may lead to situations where openpilot lateral is controlling steer while the HUD is not being updated
Verification
Changed "CC.enabled" in carcontroller and in both VW can files Confirmed desired fix with FrogPilot and feature segregation, LKA HUD LED stays illuminated/controlled while openpilot steers
Route
Route with update: 578742b26807f756/2024-03-16--19-53-30
Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:
- Convert your PR to a draft unless it's ready to review
- Read the contributing docs
- Before marking as "ready for review", ensure:
- the goal is clearly stated in the description
- all the tests are passing
- the change is something we merge
- include a route or your device' dongle ID if relevant
just wanted to add a comment to clarify, this isnt for the purpose of feature segregation. this is for covering edge cases where openpilot may be commanding steering while not indicating as such on the cluster
@jyoung8607 it is undecided for other ports whether to use latActive or enabled, what do you think, is this distracting to change when going into standstill?
@jyoung8607 it is undecided for other ports whether to use latActive or enabled, what do you think, is this distracting to change when going into standstill?
I think this is probably a good change. It will be a change for it to disappear at standstill, but I'm okay with it. In retrospect, it's a bug for us to display the green LKA indicator for cars with >0 minSteerSpeeds, and we've taken on a couple of those since this code was written.
Full disclosure while we're discussing these lines, going to yellow on steer override isn't stock behavior, it was my personal choice. It was a nice debugging aid back in the day, I grew to like it and kept it.