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

Honda: Third-gen facelift Acura MDX needs a separate car port #1637

Open
1 of 4 tasks
jyoung8607 opened this issue Jan 14, 2025 · 3 comments
Open
1 of 4 tasks

Honda: Third-gen facelift Acura MDX needs a separate car port #1637

jyoung8607 opened this issue Jan 14, 2025 · 3 comments

Comments

@jyoung8607
Copy link
Collaborator

jyoung8607 commented Jan 14, 2025

The Acura MDX got a facelift for model-year 2022, which came with a new camera/radar architecture. This architecture is also shared by newer Honda Odyssey.

Correctly supporting these cars requires a new model port with minor platform changes. The CAN messaging is nearly identical, but there are architectural changes that affect the minimum speed for lateral control and cluster display of the cruise setpoint. A port was started in commaai/openpilot#25481 by @csouers but wasn't quite completed or merged.

In the meantime, starting with commaai/openpilot#27812, we inadvertently started recognizing them as pre-facelift cars. So we need the MY2022 port and then we need to disentangle all the firmware that was added to ACURA_RDX_3G. This will be more than just reverting the one PR, the fuzzy fingerprinting bot may have added more on top.

@mvl-boston
Copy link

Thank you, for merging. Let me know if you would like me to propose code for this. In the past week I have looked through all the items mentioned and can follow everything except the fuzzy logic, and can support as needed.

There is also a problem with the longitudinal acceleration/develeration tuning parameters and I do not know if that is unique to the 2022 RDX or also impacts other bosch models.

@mvl-boston
Copy link

Issue 1638 links to the RDX information.

The MDX is a separate Acura vehicle that has its own separate incompatibility issues. The subject line for this issue and linked issue #1638 should say RDX.

@jyoung8607
Copy link
Collaborator Author

My mistake, I'll sort it out shortly. Thanks!

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

No branches or pull requests

2 participants