jyoung8607 / openpilot

open source driving agent
MIT License
73 stars 54 forks source link

Move to dynamic selection of safety model #5

Closed jyoung8607 closed 5 years ago

jyoung8607 commented 5 years ago

Panda safety model has to be hard coded today. There is something we're missing in the dynamic selection of safety profiles. Now that the port is up and running and stable, we need to revert the hard coding / all-live changes to see what we're missing.

jyoung8607 commented 5 years ago

Resolved in my most recent vw-062-base branch, safety selection is working correctly.

Recent Panda code seems to revert to NOOUTPUT safety when EON is disconnected, and we don't re-fingerprint fast enough on next start, so we're having to ignition on-off-on every time EON or OP comes back up, but that is a different issue entirely.