-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Add support for 2017 Civic Hatch 1.0L Manual (European) #535
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
openpilot 0.3.4
openpilot v0.3.5 release
openpilot v0.3.6
openpilot v0.3.6.1
openpilot v0.3.7
openpilot v0.3.8.2
openpilot v0.3.9
openpilot v0.4.0.1
openpilot v0.4.0.2
openpilot v0.4.1
openpilot v0.4.1 hotfix
Taiwanese Prius Prime fingerprint was identical to the regular prime (107 msgs.), but should really have 110. I have the correct one in there now.
* Honda Pilot 2017 Port (commaai#161) * Update README.md * Update fingerprints.py * Update carstate.py * Update hondacan.py * Update interface.py * Update interface.py * Update interface.py * Update README.md * Update README.md * Update README.md * Update fingerprints.py * Update carstate.py * Update hondacan.py * Update interface.py * Update carstate.py * Update hondacan.py * Update README.md * Update fingerprints.py * Update carstate.py * Update carstate.py * Update carstate.py * Update hondacan.py * Update interface.py * Update carstate.py * Update carstate.py * Update Pilot Fingerprint * Update fingerprints.py * Give pilot its own definition and not use ILX * add pilot argument * Add Pilot interface * Add pilot argument * Update interface.py * Parse Different gear on pilot * Add steer max * Fixed duplication of steer max value * Adjust PID's for steering * Update carcontroller.py * Change Steer Ratio and wheelbase * Update Steer fault values Steer fault value of 3, does not seem to effect anything * Update Kp,Ki Ratio * Update interface.py * Update readme for Pilot * add pilot * Update fingerprints.py * Update carstate.py * add signals * add signal * fix restricting video upload to wifi * Dibs on SAFETY_GM numerical value To match Panda repo. * Safety Reference for Honda Bosch * Update fingerprints.py (commaai#210) Consolidated my fingerprint and removed duplicates and ordered the fingerprint for the RAV4H. Double Checked. * Interpolate ki/kp for steering PID loop (commaai#200) * Interpolate ki/kp for steering PID loop Very much needed for the Volt port: car ping-pongs with low kp on high speeeds, and the loop is unstable with high kp on low speeds. Also, removes "number or array?" logic from PIController, now that all the callers use interpolation ofr ki/kp. * Pass speed to steering PID loop for ki/kp interpolation * Remove unused numbers import * Slight changes to UI and Fingerprint for Odyssey Elite (commaai#196) * Adding back drive time to UI * Add fingerprint for Odyssey Elite * Removed extended fingerprint for Elite * Revert "Adding back drive time to UI" This reverts commit b9b02f7. * Squashed 'panda/' changes from 98f29a4..67d5208 67d5208 fix signedness issue in toyota safety fe15d3f bump pandacan 11c2b08 add fault invalid 2c26e45 add sleep 27c7637 forgot the counter 3a6d7db don't hang bfa7d2e canloader works b259e2a can flasher is close to working 83f2edf isotp can support in softloader 7ae7c79 typo e85cc47 forgot the selfs 190b4f6 start work on canflasher 5c655c9 add recover support ae3457f usbflash is reliable f7a0ab0 pedal usbflash works 585d0f9 add way to call isotp be82899 despite it being bad code, move isotp 000715b start work on pedal canloader 626e312 pedal has a bootstub now 3662d1e redundant check 81e6b0d fix bug 083cd12 should have bounty to refactor that ish b65d30c bad asserts b2e6c3f isotp untested support for subaddr 30fd66a Merge pull request commaai#93 from vntarasov/volt 06f5109 Merge pull request commaai#94 from gregjhogan/can-printer-hex c7d098c Merge pull request commaai#95 from gregjhogan/setup-script 22fe250 Merge pull request commaai#99 from gregjhogan/bit-transition-example ba16ba3 Merge pull request commaai#100 from gregjhogan/j2534-troubleshooting-instructions ad08ea4 Merge pull request commaai#90 from gregjhogan/can-forwarding f3b6f5d added j2534 troubleshooting instructions 858d150 added script to find bits that transition from 0 to 1 c6acac8 added checking pedal interceptor message length f7226ff added brake safety checks d0c2634 added gas safety checks d378e4a removed bosch safety forwarding restriction on 29 bit addresses 5c7ef9e added bosch safety hooks and forwarding 90c64b6 add note 23de8d4 Merge pull request commaai#97 from commaai/pedal_improvements 0261641 added missing python packages b92b235 fix bytearray encode issue 2434f1c Tweak Volt's brake pedal stickiness e2f73d2 enable has a whole byte to itself d5a9e1e correct checksum f8ed9fa better names 986a14c don't alias pointers 9b8472e add watchdog support 8f0add9 handle faults 1d917f8 split gas set into 2 values, and have a fault state 1b77026 j2534 isn't alpha anymore fbcc872 Merge pull request commaai#92 from commaai/pedal 8a6f44b pedal is sending messages 08f464c python 3 bro is bad bro 9390961 kline checksum algo was broken... 3b7c33b add kline debug support aa622bc init values 631ea9f better refactor eb1fd75 add PEDAL adc sets ccaa310 don't build with usb 8d4d763 debug console works bd09883 comma pedal is building 75a29d5 Merge pull request commaai#84 from gregjhogan/j2534-hds eece37d only the panda has gmlan 9f43abe Merge pull request commaai#89 from vntarasov/volt 5364d43 Merge pull request commaai#88 from vntarasov/smaller-firmware 377a1ec bump version for descriptor fix 4fabdf0 Merge pull request commaai#87 from gregjhogan/usb-multi-packet-control 8580773 fix sending WinUSB Extended Properties Feature Descriptor 6908feb Chevy Volt safety 786a004 Enable optimization to reduce firmware size d70f43b hack to fix thinkpad 95ab1ae fixed flow control message padding bbd04d1 updated installer 62216d0 single standalone DLL for J2534 driver 5c9138d fixed 11 bit address issue f3b0ad2 fix LOOPBACK getting set when DATA_RATE is set b750d36 updated README a9a097f lowered CPU utilization 7c26a70 TIS needs unsupported protocols to return an error 42692b4 TIS doesn't like ChannelID being zero cf126bb SET_CONFIG return error for reserved parameters 2e99dbf fix HDS issues 8203cc8 add is_grey e946a54 add insecure_okay flag 4363b3e check webpage 4f59ded add secure mode note 6b11fb5 add autosecuring to tests b27d185 Merge pull request commaai#86 from commaai/better_pairing 4b53b42 elm wifi isn't an automated test 99f85cb Merge pull request commaai#85 from gregjhogan/usb-wcid 0d38060 auto-install WinUSB device driver c6653ca from python import 38cc0ee add wifi_secure_mode, boots in insecure mode git-subtree-dir: panda git-subtree-split: 67d5208 * Squashed 'opendbc/' changes from 81d9871..aa067f7 aa067f7 Chevy Volt tweaks (commaai#83) a60c6c4 Toyota: change signal name in EPS_STATUS msg ce70b1a extra setme field toyota LKAS_HUD df2a552 toyota missing ACC_CONTROL checksum 48bb293 Revert "Toyota Prius: added a comma specific message to control the speed sent to the EPS" 5f42439 Toyota Prius: added a comma specific message to control the speed sent to the EPS 6f5e8b6 Pedal Interceptor: fault state VAL moved to _comma efd5f5c add setme to honda ACC_HUD 97fc335 add interceptor to civic 6f40f16 update generator script to allow for multiple imports 9ca956b add setme to toyota STEERING_IPAS e5afa57 run generator for ipas scaling 8bd1182 Toyota IPAS: proper steer angle unit f57511e acceleration pedal for gasPressed c8d1dbc high beams also. likely dashboard message. 9f1c78b high beams for genericToggle f037d42 turn signal lights (and thus hazard lights) b35bb08 turn signals 78986cf Revert "turn signals" ba946c9 turn signals 2af3ecc Speed, braking, and distance signals f40ab87 Set packet lengths, adding steering rate, adjusted speed cd59bfa units for speed_right c2fcce2 speed of right vs left side of car 4ef5fae value table for gear status 97c48e2 tighten up speed bits. brake pressue max comment. a0cbfd1 add gear status PRNDL 0c82865 initial signals for chrysler pacifica 2017 hybrid 5ed0540 add set me to toyota LKAS_HUD aecac5d add set me fields to toyota ACC_HUD 5417013 update toyota ACC_CONTROL fields e91e967 Comma Pedal: made GAS_COMMAND 6 bytes d04434a Comma Pedal: added state byte and enable bit c30b2cd Comma Pedal: sending 2 tracks on 0x200 8f72467 Volt doors and belts status (commaai#70) 60f8b6c add set me to lkas hud honda 3c9e335 fix honda pcm gas message size 7ca471d Add 2018 Toyota CHR dbc (commaai#78) 637fe00 set scaling to 1 for brake and gas which have no real unit 62a88d4 Volt: switch to parsing ACC buttons from powertrain CAN (commaai#74) 3fdd47b Volt's gas pedal only and combined gas/acc (commaai#76) 45ec9c9 Add 2017 Honda Ridgeline (commaai#77) cbd186a Add 2018 Camry Hybrid DBC's (commaai#73) 974eeaf Toyota: re-generated the files after cfbc9ae 19ea195 Toyota: more vals for LKA_STATE cfbc9ae fixed inconsistent factor for speed in Honda dbc files e7db803 convert all line endings to unix style git-subtree-dir: opendbc git-subtree-split: aa067f7 * openpilot v0.4.3 release * Squashed 'panda/' changes from 67d5208..3125232 3125232 bump version 703c0b4 Gas Interceptor: another fix to gas pressed logic 196d383 Interceptor: fixed gas pressed logic git-subtree-dir: panda git-subtree-split: 3125232 * Squashed 'opendbc/' changes from aa067f7..91e882d 91e882d Updating bosch dbcs to use new format and bringing in new honda changes (commaai#82) 9b32e2e Fix Checksum errors for CH-R (commaai#86) git-subtree-dir: opendbc git-subtree-split: 91e882d * openpilot v0.4.3 release * openpilot v0.4.3.1 release * fix bug in canpacker for Toyotas with DSU connected (commaai#221) * update year on civic
This reverts commit 640ab12.
Revert "openpilot 0.4.3.1"
openpilot 0.4.5.1 release
0.4.6 Release
openpilot 0.4.7.1
openpilot 0.4.7.2
0.5.0 release
Merge commit '2cfdc676101c387524246f789c8429647069b827' into release2
Release 0.5.4
* Revert 0.5.4 changes for GM for 18ers * Redo the refactor of stock control msgs * Fixed missing CONTROL_MSGS -> STOCK_CONTROL_MSGS * Remove spacing * Need candidate array idx
0.5.5 release
Improve Toyota radar filtering (commaai#409)
calibration improvements and devel synch up
0.5.6 release
pd0wm
pushed a commit
that referenced
this pull request
May 29, 2020
b826734 Add pre commit checks + CI (#545) 339976c document tx message addresses better (#543) a618e64 fix typing errors 9bece64 use mazda init 08db086 mazda cleanup 89658d0 Mazda: safety tests add missing safety checks (#525) bdec139 Fix length of 0x20b in NISSAN_TX_MSGS, wasn't cancelling ACC (#544) b48c74c Adding UNO to automated tests (#538) a5802cd Hyundai: remove unused message from RX checks 9ebde25 Reset state on safety mode init (#542) d4f3f15 Refactor addr check (#541) 5210e51 remove unused files 0657064 Hyundai checksum (#540) 07e668e Fast CI (#539) 5307bf7 Fix multi message iso tp requests 0610ed1 Hyundai wheel speed counter is actually 4 bits spread over two signals 0d581aa dockerfile optimization eaefa2f fix docker file path 243a65f pull base image 0dd9470 only push to dockerhub from master 55b79b4 GitHub Actions (#535) b2c720b Dos (#533) 01bf740 remove 0x1BE checksum test 0bd06c9 remove 0x1BE check (breaks some vehicles) c31b899 honda bosch longitudinal safety 66250c4 Disable docker layer caching (#534) 6b19fa4 include nissan safety in release build db31886 gate mazda safety behind debug flag e4558c0 Safety: message length check on RX and TX (#529) git-subtree-dir: panda git-subtree-split: b826734
pd0wm
pushed a commit
that referenced
this pull request
May 31, 2020
49ffbe9 disable non-universal checks in hyundai safety 3a85f4c use whole route when running safety replay from CLI 098f47a Fix leaf brake rx check (#547) b826734 Add pre commit checks + CI (#545) 339976c document tx message addresses better (#543) a618e64 fix typing errors 9bece64 use mazda init 08db086 mazda cleanup 89658d0 Mazda: safety tests add missing safety checks (#525) bdec139 Fix length of 0x20b in NISSAN_TX_MSGS, wasn't cancelling ACC (#544) b48c74c Adding UNO to automated tests (#538) a5802cd Hyundai: remove unused message from RX checks 9ebde25 Reset state on safety mode init (#542) d4f3f15 Refactor addr check (#541) 5210e51 remove unused files 0657064 Hyundai checksum (#540) 07e668e Fast CI (#539) 5307bf7 Fix multi message iso tp requests 0610ed1 Hyundai wheel speed counter is actually 4 bits spread over two signals 0d581aa dockerfile optimization eaefa2f fix docker file path 243a65f pull base image 0dd9470 only push to dockerhub from master 55b79b4 GitHub Actions (#535) b2c720b Dos (#533) 01bf740 remove 0x1BE checksum test 0bd06c9 remove 0x1BE check (breaks some vehicles) c31b899 honda bosch longitudinal safety 66250c4 Disable docker layer caching (#534) 6b19fa4 include nissan safety in release build db31886 gate mazda safety behind debug flag e4558c0 Safety: message length check on RX and TX (#529) git-subtree-dir: panda git-subtree-split: 49ffbe9
pd0wm
pushed a commit
that referenced
this pull request
May 31, 2020
9102c16 two spaces before inline comment 49ffbe9 disable non-universal checks in hyundai safety 3a85f4c use whole route when running safety replay from CLI 098f47a Fix leaf brake rx check (#547) b826734 Add pre commit checks + CI (#545) 339976c document tx message addresses better (#543) a618e64 fix typing errors 9bece64 use mazda init 08db086 mazda cleanup 89658d0 Mazda: safety tests add missing safety checks (#525) bdec139 Fix length of 0x20b in NISSAN_TX_MSGS, wasn't cancelling ACC (#544) b48c74c Adding UNO to automated tests (#538) a5802cd Hyundai: remove unused message from RX checks 9ebde25 Reset state on safety mode init (#542) d4f3f15 Refactor addr check (#541) 5210e51 remove unused files 0657064 Hyundai checksum (#540) 07e668e Fast CI (#539) 5307bf7 Fix multi message iso tp requests 0610ed1 Hyundai wheel speed counter is actually 4 bits spread over two signals 0d581aa dockerfile optimization eaefa2f fix docker file path 243a65f pull base image 0dd9470 only push to dockerhub from master 55b79b4 GitHub Actions (#535) b2c720b Dos (#533) 01bf740 remove 0x1BE checksum test 0bd06c9 remove 0x1BE check (breaks some vehicles) c31b899 honda bosch longitudinal safety 66250c4 Disable docker layer caching (#534) 6b19fa4 include nissan safety in release build db31886 gate mazda safety behind debug flag e4558c0 Safety: message length check on RX and TX (#529) git-subtree-dir: panda git-subtree-split: 9102c16
pd0wm
pushed a commit
that referenced
this pull request
Jun 1, 2020
d7f7b14 Enable almost all Flake8 checks (#548) 3d5a717 fix whitespace after opening bracket f70ef29 whitespace fix eba113c backslash is redundant between brackets 8039638 fix linter complaint about unexpected spaces around keyword / parameter equals 275e76c fix linter complaint about too many blank lines 9102c16 two spaces before inline comment 49ffbe9 disable non-universal checks in hyundai safety 3a85f4c use whole route when running safety replay from CLI 098f47a Fix leaf brake rx check (#547) b826734 Add pre commit checks + CI (#545) 339976c document tx message addresses better (#543) a618e64 fix typing errors 9bece64 use mazda init 08db086 mazda cleanup 89658d0 Mazda: safety tests add missing safety checks (#525) bdec139 Fix length of 0x20b in NISSAN_TX_MSGS, wasn't cancelling ACC (#544) b48c74c Adding UNO to automated tests (#538) a5802cd Hyundai: remove unused message from RX checks 9ebde25 Reset state on safety mode init (#542) d4f3f15 Refactor addr check (#541) 5210e51 remove unused files 0657064 Hyundai checksum (#540) 07e668e Fast CI (#539) 5307bf7 Fix multi message iso tp requests 0610ed1 Hyundai wheel speed counter is actually 4 bits spread over two signals 0d581aa dockerfile optimization eaefa2f fix docker file path 243a65f pull base image 0dd9470 only push to dockerhub from master 55b79b4 GitHub Actions (#535) b2c720b Dos (#533) 01bf740 remove 0x1BE checksum test 0bd06c9 remove 0x1BE check (breaks some vehicles) c31b899 honda bosch longitudinal safety 66250c4 Disable docker layer caching (#534) 6b19fa4 include nissan safety in release build db31886 gate mazda safety behind debug flag e4558c0 Safety: message length check on RX and TX (#529) git-subtree-dir: panda git-subtree-split: d7f7b14
pd0wm
pushed a commit
that referenced
this pull request
Jun 1, 2020
d7f7b14 Enable almost all Flake8 checks (#548) 3d5a717 fix whitespace after opening bracket f70ef29 whitespace fix eba113c backslash is redundant between brackets 8039638 fix linter complaint about unexpected spaces around keyword / parameter equals 275e76c fix linter complaint about too many blank lines 9102c16 two spaces before inline comment 49ffbe9 disable non-universal checks in hyundai safety 3a85f4c use whole route when running safety replay from CLI 098f47a Fix leaf brake rx check (#547) b826734 Add pre commit checks + CI (#545) 339976c document tx message addresses better (#543) a618e64 fix typing errors 9bece64 use mazda init 08db086 mazda cleanup 89658d0 Mazda: safety tests add missing safety checks (#525) bdec139 Fix length of 0x20b in NISSAN_TX_MSGS, wasn't cancelling ACC (#544) b48c74c Adding UNO to automated tests (#538) a5802cd Hyundai: remove unused message from RX checks 9ebde25 Reset state on safety mode init (#542) d4f3f15 Refactor addr check (#541) 5210e51 remove unused files 0657064 Hyundai checksum (#540) 07e668e Fast CI (#539) 5307bf7 Fix multi message iso tp requests 0610ed1 Hyundai wheel speed counter is actually 4 bits spread over two signals 0d581aa dockerfile optimization eaefa2f fix docker file path 243a65f pull base image 0dd9470 only push to dockerhub from master 55b79b4 GitHub Actions (#535) b2c720b Dos (#533) 01bf740 remove 0x1BE checksum test 0bd06c9 remove 0x1BE check (breaks some vehicles) c31b899 honda bosch longitudinal safety 66250c4 Disable docker layer caching (#534) 6b19fa4 include nissan safety in release build db31886 gate mazda safety behind debug flag e4558c0 Safety: message length check on RX and TX (#529) git-subtree-dir: panda git-subtree-split: d7f7b14
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This also starts support for manual transmission cars. Not checking for clutch in or out, as stock acc currently handles this. When engaged and the driver needs to shift, ACC stays engaged for 3-5 seconds while the drivers shifts and re-engages the clutch. Beyond that period, it auto cancels and the hud tells the driver why it cancelled. The new Accord manual does have some bits for clutch in and out, but I don't recall where they are. This car likely does as well.
If we need to have the clutch mapped out, we can make that happen with a new error on EON as well. Safety wise, it's not crucial with stock acc.
Getting a final test on this today/tomorrow. Comments/suggestions are welcome.