From f047963f8a30e6341ffe15d34984482de3c06ef9 Mon Sep 17 00:00:00 2001 From: Jason Wen Date: Sat, 4 Jan 2025 19:27:53 -0500 Subject: [PATCH] default to true for HKG main cruise toggle --- common/params.cc | 1 - sunnypilot/mads/helpers.py | 5 +++-- system/manager/manager.py | 1 - 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/common/params.cc b/common/params.cc index 22b8c5f119..704743734f 100644 --- a/common/params.cc +++ b/common/params.cc @@ -203,7 +203,6 @@ std::unordered_map keys = { // sunnypilot params {"EnableGithubRunner", PERSISTENT}, - {"HyundaiLongitudinalMainCruiseToggleable", PERSISTENT}, {"MadsMainCruiseAllowed", PERSISTENT}, {"MadsPauseLateralOnBrake", PERSISTENT}, {"MadsUnifiedEngagementMode", PERSISTENT}, diff --git a/sunnypilot/mads/helpers.py b/sunnypilot/mads/helpers.py index 02f4c2f317..2a5d43ea52 100644 --- a/sunnypilot/mads/helpers.py +++ b/sunnypilot/mads/helpers.py @@ -51,12 +51,13 @@ class MadsParams: def set_car_specific_params(self, CP): if CP.carName == "hyundai": # TODO-SP: This should be separated from MADS module for future implementations - hyundai_cruise_main_toggleable = self.read_param("HyundaiLongitudinalMainCruiseToggleable") + # Use "HyundaiLongitudinalMainCruiseToggleable" param + hyundai_cruise_main_toggleable = True if hyundai_cruise_main_toggleable: CP.sunnypilotFlags |= HyundaiFlagsSP.LONGITUDINAL_MAIN_CRUISE_TOGGLEABLE.value CP.safetyConfigs[-1].safetyParam |= Panda.FLAG_HYUNDAI_LONG_MAIN_CRUISE_TOGGLEABLE # MADS is currently not supported in Tesla due to lack of consistent states to engage controls - # TODO-SP: To enable MADS for Tesla, dentify consistent signals for MADS toggling + # TODO-SP: To enable MADS for Tesla, identify consistent signals for MADS toggling if CP.carName == "tesla": self.params.remove("Mads") diff --git a/system/manager/manager.py b/system/manager/manager.py index 080c5c686f..ddae445e14 100755 --- a/system/manager/manager.py +++ b/system/manager/manager.py @@ -49,7 +49,6 @@ def manager_init() -> None: ("MadsMainCruiseAllowed", "1"), ("MadsPauseLateralOnBrake", "0"), ("MadsUnifiedEngagementMode", "1"), - ("HyundaiLongitudinalMainCruiseToggleable", "1"), ] if params.get_bool("RecordFrontLock"):