Honda: add fwdRadar FW for 2018 Accord Hybrid - #3663
Conversation
There was a problem hiding this comment.
Thanks for contributing to opendbc! In order for us to review your PR as quickly as possible, check the following:
- Convert your PR to a draft unless it's ready to review
- Read the contributing docs
- Before marking as "ready for review", ensure:
- the goal is clearly stated in the description
- all the tests are passing
- include a route or your device' dongle ID if relevant
Car behavior reportReplays driving segments through this PR and compares the behavior to master. 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 |
|
Comma typically wants a testroute to validate. Can you add one? |
✅ Added in the PR description |
Summary
36802-TWA-A220toCAR.HONDA_ACCORD.Root cause
The vehicle previously identified exactly as
HONDA_ACCORDwith:36161-TWA-A07036802-TWA-A070After the dealer update, the camera version was unchanged and the radar reported
36802-TWA-A220. Since A220 was absent from the database, firmware matching returned no candidate and openpilot displayed “Car not recognized.”Adding A220 produces one exact firmware match:
HONDA_ACCORD.Impact
This changes identification data only. It does not modify Honda controls, CAN parsing, or safety behavior.
Validation
07690036205f940807690036205f9408/000009f8--7aa50954ac041f324542819de25108ee16d19c24dbf4a22ae0, which contains this PR's exact opendbc change.HONDA_ACCORD, safety modelhondaBosch, forward camera36161-TWA-A070, and forward radar36802-TWA-A220../test.sh— 4,011 tests passed, 703 skipped; all lint, type, safety, and unit-test checks passed.HONDA_ACCORD.