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

ACC on Non-Sensing 2016 Civic EX-L? #50

Closed
Krishtof-Korda opened this issue Feb 4, 2017 · 5 comments
Closed

ACC on Non-Sensing 2016 Civic EX-L? #50

Krishtof-Korda opened this issue Feb 4, 2017 · 5 comments

Comments

@Krishtof-Korda
Copy link

I just bought the 2016 Civic EX-L, without Honda Sensing package. I think it would be really cool to add ACC to this car for my wife. Are there enough messages on the CAN bus to make this happen or have they been disabled on non-sensing vehicles. I really only need an 'apply brake = 0.xx' and 'apply accelerator = 0.xx' message to accomplish what I am trying to do. I am not familiar yet with the CAN bus. What would I need to start sniffing the CAN and see if the appropriate messages are available?

If possible, I plan to build the kit and do some testing. I just wanted to ask those who have experience with this before dropping the coin to build the Comma One.

Thanks for any help anyone can provide. I really think this would be a fun project and a way I could contribute to expanding capabilities for openpilot on non-sensing cars.

@Krishtof-Korda Krishtof-Korda changed the title ACC on Non-Sensing 2016 Civic EX-L ACC on Non-Sensing 2016 Civic EX-L? Feb 4, 2017
@geohot
Copy link
Contributor

geohot commented Feb 4, 2017

Here's the problem:

https://www.hondapartsnow.com/genuine/honda~motor~eps~service~53651-tba-a21.html is the steering controller from a non Honda Sensing trim.
https://www.hondapartsnow.com/genuine/honda~motor~eps~service~53651-tba-a31.html is the steering controller from the same trim with sensing.

Broken out better here, 7 variants! http://parts.sonshonda.com/showAssembly.aspx?ukey_assembly=1632831&ukey_make=988&ukey_model=13654&modelYear=2016&searchString=motor

I doubt the non sensing controller accepts the apply steer torque command. Possible to fix with a reflash maybe, but this could be a rabbit hole. You are also missing a radar, which you'd have to add.

@Krishtof-Korda
Copy link
Author

If I am not looking to steer the vehicle but just do accelerator and brake, do you think that could work with the OnePlus3 vision only at least for experimenting and testing? Then later maybe I could add the radar.

Also do you have a link for good resources on were to get smart on CAN bus and sniffing?

Thanks again for any help.

@geohot
Copy link
Contributor

geohot commented Feb 4, 2017

Same problem as with the steering for brakes, and actually it's probably even a hardware change this time and worse.

http://parts.sonshonda.com/showAssembly.aspx?ukey_assembly=1633703&ukey_make=988&ukey_model=13654&modelYear=2016&ukey_driveLine=5541&searchString=vsa

TBA variant = no auto brake
TBC variant = auto brake, and actually a very nice controller

@leo333
Copy link

leo333 commented May 22, 2017

@geohot
How do you start the ACC module in the car after the radar is added?

rbiasini pushed a commit that referenced this issue Nov 17, 2018
3f6cd74 Add 3rd RSA signal and cleanup (#120)
a308ec1 add vals honda (#121)
1317514 Toyota: fixed typos
0339fc8 Toyota: added a better cruise active indicator
b0c012d Lexus: add is
94ba43c Toyota: generated dnc files after latest change
e16723e Add Distance Lines and RSA (#118)
9aec1ef Add GM signals for ESP/Mode/Highbeams/Intellibeam (#114)
a67fff9 add   toyota_prius_2010_pt.dbc (#50)
c1ee10f Revert "Revert "GM: copy radar header from cadillac (#116)""

git-subtree-dir: opendbc
git-subtree-split: 3f6cd74
SippieCup pushed a commit to SippieCup/openpilot that referenced this issue May 19, 2019
…pdates-0.5.11

Config defaults updates 0.5.11 (+ unit tests)
SippieCup pushed a commit to SippieCup/openpilot that referenced this issue May 19, 2019
@pd0wm pd0wm closed this as completed Jun 2, 2019
avolmensky pushed a commit to avolmensky/openpilot that referenced this issue Jun 10, 2019
* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* OpenDBC updates

* Revert "OpenDBC updates"

This reverts commit cb3a4a8ae6f5fc36e3d1b8cd80d3f5b27fb39b9f.

* Delete .gitignore

* Delete toyota_prius_2010_pt.dbc

* Rename toyota_prius_2010_cam.dbc to toyota_prius_2010_pt.dbc

* Revert "Delete .gitignore"

This reverts commit 47f60f3c5e0207d280c1b9aac83791d49ac7eb78.

* remove prius 2010 dbc from gitignore
avolmensky pushed a commit to avolmensky/openpilot that referenced this issue Jun 30, 2019
jordanmkoncz referenced this issue in jordanmkoncz/openpilot Jan 27, 2020
mcomg pushed a commit to mcomg/openpilot---jpancotti that referenced this issue Jun 27, 2020
debugged-tech referenced this issue in debugged-tech/DebuggedPilot Jul 22, 2020
plannerError does not need to disengage
weimou pushed a commit to SFtoLA/openpilot that referenced this issue May 6, 2021
* fix mac build

* remove duplicate line

Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com>
twilsonco pushed a commit to twilsonco/openpilot that referenced this issue Feb 17, 2022
jbms pushed a commit to jbms/openpilot that referenced this issue Dec 28, 2023
fix: nnff model must exact match car (eps can be fuzzy)
twilsonco pushed a commit to twilsonco/openpilot that referenced this issue Mar 10, 2024
* golf varient haxs (ACC equipped) (commaai#49)

* golf varient haxs (ACC equipped)

* re-enable UDS FW query on ign

* remove min steer speed, oops
carleeno pushed a commit to carleeno/openpilot that referenced this issue Aug 2, 2024
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

No branches or pull requests

4 participants