Tim Wilson

Results 85 comments of Tim Wilson

Here's plots also showing the button events saved to car state for the segments A and B ```cpp # send on change struct ButtonEvent { pressed @0 :Bool; type @1...

So neither the CAN ACC button press for res/accel or the cereal car state res/accel button press is being set in a way that corresponds to the set speed increases....

I notice that the CS.buttonEvents is only set if the conditional on line 170 of gm/interface.py runs, causing most frames to not have any data about the unpressed status of...

If the problem isn't something to do with ret.buttonEvent not being set each iteration, then it should be in controls or drive helpers: ![image](https://user-images.githubusercontent.com/7284371/184509635-32781d81-3749-4920-9a19-e7c2648ae3fc.png) ![image](https://user-images.githubusercontent.com/7284371/184509664-2b0d73fd-76c8-4c97-b767-1c725c26cc71.png) ![image](https://user-images.githubusercontent.com/7284371/184509675-ba17500b-c813-4359-9386-0cd07e212db0.png)

I'm speculating but it seems to be typical for GM ACC, so I would think it applies to all GM.

![image](https://user-images.githubusercontent.com/7284371/186072113-743cfbab-3c77-4ca1-9729-1980be88d6c3.png) Okey dokey!

Installing. I'll test in the morning. Looks like I'll be pressing the resume button in different scenarios. Stopped, engaging, increase set speed, resume behind lead?

Ack! That explains all the `CAMERA` I was seeing!

It's more of a sidenote that the torque controller was already fetching the other (traditional) custom FF functions. The point here is that this should be added to improve performance...

~~Shall I add the Volt and Acadia torque FF functions to this PR with routes? The Acadia route will be recorded on a C2 with commatwo_master; the Volt will use...