Skip to content

Added EPB and Brake_Hold - #61

Merged
rbiasini merged 2 commits into
commaai:masterfrom
joeljacobs:master
Dec 21, 2017
Merged

rbiasini merged 2 commits into
commaai:masterfrom
joeljacobs:master

Conversation

@joeljacobs

Copy link
Copy Markdown
Contributor

Searched through Cabana with @energee to find proper signals for EPB and Brake_Hold. We found them. 

Comment thread honda_odyssey_exl_2018.dbc Outdated
SG_ CHECKSUM : 51|4@0+ (1,0) [0|3] "" EON

BO_ 450 XXX_6: 8 XXX
SG_ EPB_Brake_and_Pull_Indicator : 6|1@0+ (1,0) [0|1] "" XXX

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@joeljacobs can you use CAPITAL notation here too? other than that it looks good

@rbiasini
rbiasini merged commit 0d3aea0 into commaai:master Dec 21, 2017
chrispypatt pushed a commit to chrispypatt/opendbc that referenced this pull request Feb 2, 2025
* add json

* empty to test

* add generator

* generated

* add print

* more abstract

* rename

* generated

* add test to ensure list always syncs with the docs

* attempt to fail the test

* fix path

* correct path

* fail test intentionally

* Revert "fail test intentionally"

This reverts commit bf456ba.

* start clean

* dump back

* update name

* new list

* generate extra list

* single list is fine

* more precision search

* set while initializing

* tldr

* generate `car.CarParams.brand` in json

* unused

* generate no car docs platforms

* include all dashcamOnly platforms

* do not include dashcamOnly yet
eli-davis pushed a commit to eli-davis/opendbc_reference that referenced this pull request Jul 15, 2025
* Added EPB and Brake_Hold

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants