openpilot_comma

329 Commits

93 Branches

84 Tags

7.1 GiB

Tag: Branch: Tree: 4a94e507cf

Commit Graph

Author	SHA1	Message	Date
Adeeb	13f60eae91	Abstract CarInterface init (#1131 ) * abstract CarInterface init * unused * subaru * gm cleanup * vw * typo * carcontroller * more cleanup * fix vw	6 years ago
andrewcopenpilot	4b12aca941	GM: Resume Required Alert should be triggered off of ECM state (#1045 ) * Resume Required Alert should be triggered off of ECM state On GM cars the ECM goes into cruise state 4 (standstill) and in order for the ECM to accept cruise inputs, the resume button must be pressed to drive it to state 1 (active). Previously rather than testing for if the ECM was in state 4, the wheel speed sensors were checked that the vehicle was going less than 0.5 m/s. In some cases this could cause the resume required alert to be displayed before the car actually came to a stop as far as the ECM was concerned. If the alert is triggered before state 4 and OP demands acceleration, its possible for OP to acelerate while the Resume Required Alert is displayed created a very confusing UX. * After more review, the problem was actually with the 'if at_full_stop' check was inside an else statement preventing entering the stopped mode if braking is 0	6 years ago
George Hotz	71ead9adea	selfdrive/car	6 years ago

Author

SHA1

Message

Date

Adeeb

13f60eae91

Abstract CarInterface init (#1131 )

* abstract CarInterface init

* unused

* subaru

* gm cleanup

* vw

* typo

* carcontroller

* more cleanup

* fix vw

6 years ago

andrewcopenpilot

4b12aca941

GM: Resume Required Alert should be triggered off of ECM state (#1045 )

* Resume Required Alert should be triggered off of ECM state

On GM cars the ECM goes into cruise state 4 (standstill) and in order
for the ECM to accept cruise inputs, the resume button must be pressed
to drive it to state 1 (active).

Previously rather than testing for if the ECM was in state 4, the wheel
speed sensors were checked that the vehicle was going less than 0.5 m/s.

In some cases this could cause the resume required alert to be displayed
before the car actually came to a stop as far as the ECM was concerned.
If the alert is triggered before state 4 and OP demands acceleration,
its possible for OP to acelerate while the Resume Required Alert is
displayed created a very confusing UX.

* After more review, the problem was actually with the 'if at_full_stop' check was inside an else statement preventing entering the stopped mode if braking is 0

6 years ago

George Hotz

71ead9adea

selfdrive/car

6 years ago

3 Commits (4a94e507cff94e55885e1b5efd2aa84bf642634d)