Make the time from initial wheel touch alert to making noise longer
See original GitHub issueI just took my first drive with 0.7.10 (since 0.7.7) tonight and since I don’t have DM at night I use wheel touch. However with the changes to wheel touch timings in this commit https://github.com/commaai/openpilot/commit/21d4ac1eebba06769f03edf7220ce84bd6ae154d I find myself not being able to catch the alert fast enough so that it doesn’t make noise. If I’m looking straight at the road, as soon as I notice that the alert to touch the wheel has shown, it’s just then turning to orange and starts making noise, possibly waking up anyone else in the car (but is also annoying if it’s just you).
Is there any way we can either delay this prompt noise, or revert part of that commit to make the time from initial “touch the wheel” alert to making noise alert its original length? It was perfect before, but now makes driving at night much more annoying. Another annoyance is the orange warning alert that makes noise turns on and off repeatedly if the monitoring model is uncertain, since the orange alert resets the hi_std_alert_enabled
variable, so it’s just in a loop until I touch the wheel, even if I’m looking at the road and the model can see me.
That last thing could be solved by a timeout on how quickly it can show the uncertain alert after orange alert is shown.
Issue Analytics
- State:
- Created 3 years ago
- Comments:13 (11 by maintainers)
Top GitHub Comments
Have any ideas what can be changed to fix the flickering?
I’ll go out and test the original values(/find ones that seem better) as well as try to fix that uncertain alert flickering while the orange “touch wheel” alert is on screen and open a PR for that
(Ah I see how that works now! 7-5=2 seconds for green prompt alert. Yeah that’s way too little, everyone I have try openpilot they don’t notice the screen quick enough for the older times yet alone 2 seconds lol)