see what Actions says about this

pull/20922/head
Shane Smiskol 4 years ago
parent 2d515f3721
commit c908488854
  1. 3
      selfdrive/car/honda/carcontroller.py
  2. 3
      selfdrive/car/honda/interface.py

@ -153,8 +153,7 @@ class CarController():
can_sends.append(hondacan.create_bosch_supplemental_1(self.packer, CS.CP.carFingerprint, idx)) can_sends.append(hondacan.create_bosch_supplemental_1(self.packer, CS.CP.carFingerprint, idx))
# If using stock ACC, spam cancel command to kill gas when OP disengages. # If using stock ACC, spam cancel command to kill gas when OP disengages.
if pcm_cancel_cmd: if pcm_cancel_cmd:
# can_sends.append(hondacan.spam_buttons_command(self.packer, CruiseButtons.CANCEL, idx, CS.CP.carFingerprint)) can_sends.append(hondacan.spam_buttons_command(self.packer, CruiseButtons.CANCEL, idx, CS.CP.carFingerprint))
pass
elif CS.out.cruiseState.standstill: elif CS.out.cruiseState.standstill:
can_sends.append(hondacan.spam_buttons_command(self.packer, CruiseButtons.RES_ACCEL, idx, CS.CP.carFingerprint)) can_sends.append(hondacan.spam_buttons_command(self.packer, CruiseButtons.RES_ACCEL, idx, CS.CP.carFingerprint))

@ -144,6 +144,9 @@ class CarInterface(CarInterfaceBase):
ret.enableCruise = not ret.enableGasInterceptor ret.enableCruise = not ret.enableGasInterceptor
ret.communityFeature = ret.enableGasInterceptor ret.communityFeature = ret.enableGasInterceptor
if not ret.openpilotLongitudinalControl: # make sure if car is controlling ACC that we don't have pedal, I'm not sure how Honda works
assert ret.enableCruise
# Certain Hondas have an extra steering sensor at the bottom of the steering rack, # Certain Hondas have an extra steering sensor at the bottom of the steering rack,
# which improves controls quality as it removes the steering column torsion from feedback. # which improves controls quality as it removes the steering column torsion from feedback.
# Tire stiffness factor fictitiously lower if it includes the steering column torsion effect. # Tire stiffness factor fictitiously lower if it includes the steering column torsion effect.

Loading…
Cancel
Save