Skip to content

honda: port Bosch-A 16-slot radar decoder, enable for full HONDA_BOSCH_A harness - #1

Closed
JamesL787 wants to merge 1 commit into
nrdr:nrdr-barefrom
JamesL787:nrdr-bare
Closed

JamesL787 wants to merge 1 commit into
nrdr:nrdr-barefrom
JamesL787:nrdr-bare

Conversation

@JamesL787

Copy link
Copy Markdown

Summary

  • Ports the firmware-correct 16-slot Bosch-A object bank radar decoder (from night-star-bosch-radar, base commit + the vRel-authority fix that closes a raw-derivative-poisons-KF hazard) into radar_interface.py as a branch alongside Nidec — same pattern Nidec already uses, no delegate class.
  • Wires it in for the full HONDA_BOSCH_A harness category (10 platforms: Civic Bosch, Civic Bosch Diesel, Accord, CR-V 5G, CR-V Hybrid, Acura RDX 3G, Insight, Honda e, Honda e Advance, N-Box), not just Civic Bosch.
  • Gate lives in interface.py (this branch has no CarInterface extension-point layer to hook into), keyed on a new HondaBoschARadar param (default on).

Test plan

  • 86/86 tests pass (opendbc/car/honda/tests/test_bosch_a_radar.py), verified locally on this branch's actual submodule commit
  • On-road validation across the newly-enabled platforms (only Civic Bosch has been road-tested with this exact decoder so far)

🤖 Generated with Claude Code

…H_A harness

Ports the firmware-correct 16-slot Bosch-A object bank decoder (from
night-star-bosch-radar, base + the vRel-authority fix in myfork@e49ac1d33b)
into radar_interface.py as a branch alongside Nidec -- no delegate class,
same pattern Nidec already uses.

Enables it for the full HONDA_BOSCH_A harness category (10 platforms: Civic
Bosch, Civic Bosch Diesel, Accord, CR-V 5G, CR-V Hybrid, Acura RDX 3G,
Insight, Honda e, Honda e Advance, N-Box), gated inline in interface.py by a
single HondaBoschARadar param (default on, no CarInterface extension-point
layer exists on this branch to hook into). 86/86 tests pass.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@nrdr

nrdr commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Closing as already ported and superseded on the current development line. Thank you for the original Bosch-A radar work.

This closes the obsolete PR targeting nrdr-bare; it does not claim the original commit was merged into that historical branch. Reapplying it to development would duplicate/replace newer work. No source changes were made during this reconciliation.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants