From 97b5997b8b6ea98c1cb68ce7c4a4531ba60e3b03 Mon Sep 17 00:00:00 2001 From: kegman Date: Mon, 4 Feb 2019 23:30:19 -0500 Subject: [PATCH 1/2] Eliminate brake oscillations and grinding / crunching / ripping sound when using Pedal (Honda Pilot Only) (#475) * Eliminate brake oscillations when using Pedal * Simplified code * syntax error fix * Grinding code as applied to Honda Pilot only Each model may need its own setting due to USER_BRAKE noise disengagements * correction: self.CP.carFingerprint instead of CP.carFingerprint * Add Ridgeline to Pedal Grinding fix Confirmed to work also on Ridgeline --- selfdrive/car/honda/carstate.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/selfdrive/car/honda/carstate.py b/selfdrive/car/honda/carstate.py index 97407775f..a064a0b72 100644 --- a/selfdrive/car/honda/carstate.py +++ b/selfdrive/car/honda/carstate.py @@ -303,7 +303,11 @@ class CarState(object): self.user_brake = cp.vl["VSA_STATUS"]['USER_BRAKE'] self.pcm_acc_status = cp.vl["POWERTRAIN_DATA"]['ACC_STATUS'] self.hud_lead = cp.vl["ACC_HUD"]['HUD_LEAD'] - + + # gets rid of Pedal Grinding noise when brake is pressed at slow speeds for some models + if self.CP.carFingerprint in (CAR.PILOT, CAR.PILOT_2019, CAR.RIDGELINE): + if self.user_brake > 0.05: + self.brake_pressed = 1 # carstate standalone tester if __name__ == '__main__': From d03e94c4d5d3e99d65bf0343b71fd0a873af1df7 Mon Sep 17 00:00:00 2001 From: Riccardo Date: Wed, 6 Feb 2019 16:40:08 -0800 Subject: [PATCH 2/2] Readme: FCA giraffe added --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index 4631a02b1..18b8a9402 100644 --- a/README.md +++ b/README.md @@ -65,7 +65,7 @@ Supported Cars | Chevrolet3| Malibu 2017 | Adaptive Cruise | Yes | Yes | 0mph | 7mph | Custom7| | Chevrolet3| Volt 2017-18 | Adaptive Cruise | Yes | Yes | 0mph | 7mph | Custom7| | Cadillac3 | ATS 2018 | Adaptive Cruise | Yes | Yes | 0mph | 7mph | Custom7| -| Chrysler | Pacifica Hybrid 2017 | Adaptive Cruise | Yes | Stock | 0mph | 9mph | Custom8| +| Chrysler | Pacifica Hybrid 2017 | Adaptive Cruise | Yes | Stock | 0mph | 9mph | FCA | | GMC3 | Acadia Denali 2018 | Adaptive Cruise | Yes | Yes | 0mph | 7mph | Custom7| | Holden3 | Astra 2017 | Adaptive Cruise | Yes | Yes | 0mph | 7mph | Custom7| | Honda | Accord 2018 | All | Yes | Stock | 0mph | 3mph | Bosch | @@ -103,7 +103,6 @@ Supported Cars 528mph for Camry 4CYL L, 4CYL LE and 4CYL SE which don't have Full-Speed Range Dynamic Radar Cruise Control. 6Open sourced [Hyundai Giraffe](https://github.com/commaai/neo/tree/master/giraffe/hyundai) is designed for the 2019 Sante Fe; pinout may differ for other Hyundais. 7Community built Giraffe, find more information [here](https://zoneos.com/shop/). -8Community built Giraffe, find more information [here](https://github.com/adhintz/openpilot/wiki). Comma official Giraffe coming soon. Community Maintained Cars ------