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.

Sign up for GitHub

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

Jump to bottom

GM: no speed limit to engage #373

Closed
wants to merge 2 commits into from
Closed

GM: no speed limit to engage #373

wants to merge 2 commits into from

Conversation

vntarasov
Copy link

Remove "18mph to engage" limit to engage openpilot, through catching a PCM fault and showing the same "Speed too low" error instead.
Since openpilot must be first enabled with the "set" button, this change also disables "resume" button before openpilot is first engaged with "set": "Speed too low" would be a wrong message to show in that case.

Pros:

  • re-engagement is possible at much lower speeds. For 2017 Volt:
    • both "set" and "resume" re-engagement work at any speed, if brake pedal was not pressed since disengagement.
    • if brake pedal was pressed since disengagement, resume works above ~7 mph and set works above ~18mph.
  • 2018 might have different conditions to engage. With this change, we have a precise speed limit for engagement, no need for overshooting the limit, no need to dig for PCM states for various model years.

Cons:

  • If PCM fault happens due to an unrelated bug, "Speed too low" would be a wrong message to show.
  • Different user experience compared to "wrong gear" and other known state errors: instead of showing "Speed too low" right away, PCM fault catching & message will happen split second after openpilot engages.

@vntarasov
Copy link
Author

Need a message around the lines of 'Press "set/-" first', instead of just ignoring the button, for better user experience.

@vntarasov vntarasov closed this Oct 7, 2018
pd0wm pushed a commit that referenced this pull request Dec 14, 2019
Squashed 'panda/' changes from 256d274..a648cca
bc7b9b3 
a648cca Add os import
042562d Extracted wifi connect from test helpers
ac0fd5d query fw versions example - use extended diagnostic session
4e9d788 Remove not-needed cadillac-init
f0a5d15 typo
c093286 Add bootkick after re-enabling phone power (#401)
eadb0db security upgrades (#397)
7c13bec Command to get signature (#399)
dad439a static assert on size of health packet (#398)
da9da46 Fix VERSION
df4159c Revert "Revert "Register readback on most modules. Still need to convert the other ones (#396)""
56ec215 Revert "Register readback on most modules. Still need to convert the other ones (#396)"
893e486 Register readback on most modules. Still need to convert the other ones (#396)
6bbae7b VW safety: allow cancel spam on both buses to be compatible with camera and gateway integration
d5f7a28 bump panda
1bcc351 ignition_can: set it to False after 2s of not seeing CAN msgs
96137f1 VW can based ignition not needed. it has ignition line at camera as well.
1b004a1 Same flake8 version as the one in openpilot
e82ba5f Same pylint version as the one in openpilot
656f99b Interrupt refactor (NVIC_SM_1: #334) and Fault handling (#377) (PR #373)
000282e Fix can_logger.py to run correctly on python3 (#392)
7f9b4a5 Fix USB device enumeration on Windows 8.1 and Windows 10 (#393)
dec565c Update Misra test coverage, which now includes rule 2.7
fb6bc3b Fix Misra
878dd00 solve race condition is relay_malfunction right after changing the relay status by adding a counter
2d4cb05 add a safety mode counter
a6797a21 Implement USB power mode on uno
670f90c Merge branch 'master' of github.com:commaai/panda
ca39a5d Added faults integer to health packet
e1c34a1 Panda Jungle testing (#394)
2a093a3 Added heartbeat to echo test
2246435 Fixed health struct size. We should really get an automated test for this
f458d67 Add uptime counter to the health packet (#391)
1662481 enable CAN transcievers outside the set_safety_mode function, which is not related
a7c9874 bump panda ver
1192d93 Power saving refactor (#389)
d58d08f Fix Misra 17.8: can't mod function params
bc685ac Minor indent
a54b86c Failure of set_safety_mode falls back to SILENT. Failure to set silent results in hanging
597436d NOOUTPUT safety mode is now SILENT. NOOUTPUT still exists but keeps C… (#388)
d229f8d ESP forced off in EON build. this prevents ESP to be turned on when e… (#387)
8a044b3 forgot Hyundai: now also using make_msg
4f9c879 remove abunch of lines from safety regression tests by using common make_msg function
fb81414 mispelled word
57f5ef8 Fix misra: addr can't be more than 29 bits anyway
68ff501 typo
d5c772b Fixe Toyota message white-list
48197a9 Better masking for ELM mode
b8fe78c VW is also tested for safety replay
212d336 Safety Chrysler: Added cancel spam button check
d44b562 fix print in example
02d579a functional addr handling
6249a18 tx_hook shall have a white-list of messages (#381)
8138fc1 uds: handle function addrs and fw version query example
6626a54 Fixed python health api
b9b79e8 uds zero second timeout
e0de1a4 define ALLOW_DEBUG in safety tests
86dec4b Safety modes that violate ISO26262 requirements are not compiled in RELEASE build
e74ed93 safety tests a bit more simplified
2027765 relay malfunction test centralized
8af1a01 clean up safety tests
e8f7a3b upd panda
cfcce8f WIP: Relay malfunction (#384)
69d9d61 No tabs in mazda safety
a86418c insignificant changes
f239b99 single addr was better
d063a18 Hyundai safety: re-enable button spam safety check
4d1edc0 skip tx_hook if a message is forwarded (#379)
df2ff04 bump version
168461d added fault state to health packet
b3e1a13 uds: better debug prints
68c39fb uds: no need for threads if you always drain rx
91b7c5b bump Panda Ver
26cb4dc Fixed pylint error
32725cc Fixed misra compliance
e33b4be Added echo script
312ba62 minor comment cleanupo
e90897a Fix board detection on white
0e72c18 always stop executing if safety mode fails to be set (suggested by jyoung8607)
e8d7ed1 Rename function name to not confuse safety_set_mode and set_safety_mode
ff86db6 improve uds message processing
512ab3f except Exception
37ce507 py3 all
bac4d85 dos and python3
501db8d uds drain before send and use has_obd()
f2cbec1 Added has_obd() to python library
48e5b18 Add SDK downloading to the build step (#314)
e0762c2 Add Python & USB API for controlling phone power (#313)
ba9fb69 New health packet struct also in the python libs

git-subtree-dir: panda
git-subtree-split: a648cca
Vaggysag pushed a commit to Vaggysag/openpilot that referenced this pull request Apr 21, 2020
@robbederks
Interrupt refactor (NVIC_SM_1: commaai#334) and Fault handling (comma…
656f99b 
…ai#377) (PR commaai#373)
nworb-cire pushed a commit to opgm/openpilot that referenced this pull request Aug 2, 2024
@sunnyhaibin
MADS: Map cruise main and lateral buttons to button events (commaai#373)
54d5281 
* MADS: Map cruise main and lateral buttons to button events
* MADS: Map cruise main and lateral buttons to button events

* different style

* nah

(cherry picked from commit 4d7afe3)

* reimplement
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vntarasov