Skip to content

Honda - additional comma power fingerprints for Acura RDX 3G MMR#3335

Open
mvl-boston wants to merge 2 commits intocommaai:masterfrom
mvl-boston:rdx-fp
Open

Honda - additional comma power fingerprints for Acura RDX 3G MMR#3335
mvl-boston wants to merge 2 commits intocommaai:masterfrom
mvl-boston:rdx-fp

Conversation

@mvl-boston
Copy link
Copy Markdown
Contributor

2024: 4d8ba79da0eabd91/00000031--ecbad56760
2026: 3792d010590cb83a/00000009--8b3387841b

@github-actions github-actions Bot added car related to opendbc/car/ honda fingerprint labels Apr 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Car behavior report

Replays driving segments through this PR and compares the behavior to master.
Please review any changes carefully to ensure they are expected.

Testing 154 segments for: HONDA_ACCORD, HONDA_CIVIC_BOSCH, HONDA_CIVIC_2022, HONDA_CRV_5G, HONDA_CRV_HYBRID, HONDA_HRV_3G, HONDA_INSIGHT, HONDA_E, HONDA_PILOT_4G, HONDA_CRV, HONDA_FIT, HONDA_FREED, HONDA_HRV, HONDA_ODYSSEY, HONDA_PILOT, HONDA_RIDGELINE, HONDA_CIVIC

✅ 0 changed, 154 passed, 0 errors

@sshane
Copy link
Copy Markdown
Contributor

sshane commented Apr 18, 2026

It should be able to FP without comma power though, right?

@mvl-boston
Copy link
Copy Markdown
Contributor Author

Yes all these fingerprint wo comma power already. The camera side of the gateway only exposes fwdcamera and fwdradar ecus and those IDs are the same for every year of the RDX_3G_MMR.

Would it be preferred to remove the other Ecu types instead so fingerprinting works without this inventory?

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

Labels

car related to opendbc/car/ fingerprint honda

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants