Jason Young

Results 220 comments of Jason Young

> **TL;DR:** We need to get the ACC radar FW It turns out we're getting `fwdRadar` firmware fine; it's mislabeled as `fwdCamera`. 0x753 is the ACC radar on older cars,...

I was thinking we'd see UDS engine firmware show up as the cars got newer, but nothing yet. Reverted the fancy footwork in fw_versions.py in favor of a comment for...

@j-vanetten of jvePilot fame rebased his release on 0.8.15 yesterday, and [pulled a snapshot of this PR into jvePilot](https://github.com/j-vanetten/openpilot/commits/jvePilot-release) this evening. That will put the upgraded 0.8.15 UDS code and...

Unit test added, but pulled back to draft. We've gotten a Pacifica Hybrid and it's not returning transmission or ABS/ESP FW, working to find out what's going on there. `ede4a914fcc6fb6d|2022-08-11--14-55-50`

If you're comfortable with the state of PacHy returns from 0.8.15, and you'd rather ingest all the FW from your notebook workflow than try to reconcile it with this PR,...

Since you've got your head in this tool: i always wanted to split out left and right vs combining/averaging them, to look for effects of road crown on self-centering or...

The end user here is having to have a custom windshield mount printed, hoping to have a test drive this weekend.

> @sshane is there anything I'm doing wrong in [this PR](https://github.com/commaai/openpilot/pull/26136) that would lead these checks to fail? I haven't changed CarInfo and I've only added to CarParams. I originally...

For the docs unit tests, when you removed the min steer speed from the individual HondaCarInfo instances, the default min steer speed set in the [top level HondaCarInfo class](https://github.com/commaai/openpilot/blob/826d8a8ae34bfd909535f5edff2229b8b2daf1a6/selfdrive/car/honda/values.py#L105-L108) replaced...

The explanation from jshuler is possible, but I would be somewhat surprised. If the recipient can't tolerate 50Hz (every 20ms) messages occasionally arriving 19.9ms apart, the system as a whole...