remove Pacifica 2019 from supported car list #538
Merged
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
Although there's a fingerprint and LKAS constant for the 2019 Pacifica, it doesn't work well enough that I'd consider it supported. P.S. I have an idea for how to support all Chrysler with one code base, even those I've never looked at. Forward can0 to the stock LKAS (can2) so that the stock LKAS is active. Then read the stock LKAS messages on can2 to both get the vehicle's LKAS constant, any status messages, and perhaps a more reliable counter. I'll try to code it up this weekend.
rav4kumar
pushed a commit
to rav4kumar/openpilot
that referenced
this pull request
Feb 22, 2020
dragonpilot
pushed a commit
to dragonpilot-community/dragonpilot
that referenced
this pull request
Apr 7, 2020
Although there's a fingerprint and LKAS constant for the 2019 Pacifica, it doesn't work well enough that I'd consider it supported. P.S. I have an idea for how to support all Chrysler with one code base, even those I've never looked at. Forward can0 to the stock LKAS (can2) so that the stock LKAS is active. Then read the stock LKAS messages on can2 to both get the vehicle's LKAS constant, any status messages, and perhaps a more reliable counter. I'll try to code it up this weekend.
pd0wm
pushed a commit
that referenced
this pull request
May 28, 2020
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: bdec139
pd0wm
pushed a commit
that referenced
this pull request
May 28, 2020
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: bdec139
pd0wm
pushed a commit
that referenced
this pull request
May 28, 2020
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: bdec139
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.
Although there's a fingerprint and LKAS constant for the 2019 Pacifica, it doesn't work well enough that I'd consider it supported.
P.S. I have an idea for how to support all Chrysler with one code base, even those I've never looked at. Forward can0 to the stock LKAS (can2) so that the stock LKAS is active. Then read the stock LKAS messages on can2 to both get the vehicle's LKAS constant, any status messages, and perhaps a more reliable counter. I'll try to code it up this weekend.