Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chrysler: Safety mode for Compact US Wide #1760

Open
wants to merge 55 commits into
base: master
Choose a base branch
from

Conversation

jyoung8607
Copy link
Collaborator

@jyoung8607 jyoung8607 commented Dec 7, 2023

  • Fix issue with checksums/counters in RX checks
  • Optimize factoring/sharing between this and the existing Chrysler port
  • Exit from ALLOW_DEBUG (to be done by comma after initial merge)
  • Look for a better PCM message
  • Accommodate torque ramp-down on disengage (will do this in another PR)

@jyoung8607 jyoung8607 marked this pull request as ready for review May 1, 2024 04:52
@jyoung8607
Copy link
Collaborator Author

Releasing from draft, ready for review.

This is safety-complete, but driving experience isn't great because the CUSW steering rack will sometimes fault when being slammed down to zero torque instantly on disengage. Per our Discord conversation back in February, I'm working on something to allow torque to fall at MAX_RATE_DOWN when disengaging. It's going to be a tricky change to critical code. It deserves its own PR, and can't come in first due to CI coverage tests. I propose to bring CUSW safety in now, gated under ALLOW_DEBUG like we'd probably start with anyway, and come back for the disengagement fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant