-
Notifications
You must be signed in to change notification settings - Fork 791
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 longitudinal #1719
base: master
Are you sure you want to change the base?
chrysler longitudinal #1719
Conversation
ae8df79
to
b3ea2ca
Compare
1a9442f
to
055b09e
Compare
055b09e
to
2825df6
Compare
431f8b8
to
f4cfd3d
Compare
ea499c0
to
7994f48
Compare
7994f48
to
0dddd41
Compare
0dddd41
to
961defe
Compare
961defe
to
fb5b0ae
Compare
fb5b0ae
to
f811f76
Compare
3acc20d
to
abe63a6
Compare
…hecks that it endswith Base)
…thout this todo: falsify those tests?
abe63a6
to
eb91c60
Compare
I don't have enough time to fully review, but this PR looks quite large. Is there anything that can be split out, such as the rx check macro refactor? That would make things quicker to review, hard to know what is required and what is part of a clean up. If that's the case and it's split out, then we could easily see what was added in the diff. |
There are probably several PRs you could split this up into (RX checks and essentially 3 sub-platforms). This has been open so long now that I am also guessing there are other general improvements I am unaware of that need to be incorporated, too. If this isn't mergeable and you find time to work on this maybe you just want to create new PRs and use this one as a reference. I probably won't have time to in the near future, but the tests should be quite good. |
TODO:
DAS_3.ENGINE_TORQUE_REQUEST
DAS_3.ACC_DECEL