From 24bebd4ddeac352a7b67246e572fb817e34dbfbd Mon Sep 17 00:00:00 2001 From: firestar5683 <168790843+firestar5683@users.noreply.github.com> Date: Fri, 6 Nov 2026 21:43:16 -0600 Subject: [PATCH] Qualify split HKG angle EPS publications --- .github/workflows/hkg-angle-recovery-check.yaml | 2 +- openpilot/starpilot/flm/tests/test_pid_global_capture.py | 2 +- .../starpilot/flm/tests/test_registered_controls_global.py | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/hkg-angle-recovery-check.yaml b/.github/workflows/hkg-angle-recovery-check.yaml index 3af9bc7521..bdad6e3bac 100644 --- a/.github/workflows/hkg-angle-recovery-check.yaml +++ b/.github/workflows/hkg-angle-recovery-check.yaml @@ -39,7 +39,7 @@ jobs: openpilot/selfdrive/locationd/models/generated/libcar.so - name: Run steering retention and independent safety veto regressions run: | - .venv/bin/python -m pytest -q --junitxml=qualification.xml openpilot/starpilot/aol/tests/test_hkg_angle_eps_retention.py openpilot/starpilot/aol/tests/test_pe_combined_profile_contract.py openpilot/starpilot/aol/tests/test_stock_eps_recovery.py openpilot/starpilot/aol/tests/test_runtime.py openpilot/starpilot/aol/tests/test_source_pause_driver_monitoring.py openpilot/system/hardware/tests openpilot/starpilot/drive_state/tests/test_hardware_lifecycle.py openpilot/starpilot/ui/tests/test_weather.py openpilot/starpilot/flm/tests/test_pid_global_capture.py + .venv/bin/python -m pytest -q --junitxml=qualification.xml openpilot/starpilot/aol/tests/test_hkg_angle_eps_retention.py openpilot/starpilot/aol/tests/test_pe_combined_profile_contract.py openpilot/starpilot/aol/tests/test_stock_eps_recovery.py openpilot/starpilot/aol/tests/test_runtime.py openpilot/starpilot/aol/tests/test_source_pause_driver_monitoring.py openpilot/system/hardware/tests openpilot/starpilot/drive_state/tests/test_hardware_lifecycle.py openpilot/starpilot/ui/tests/test_weather.py openpilot/starpilot/flm/tests/test_pid_global_capture.py openpilot/starpilot/flm/tests/test_registered_controls_global.py::test_actual_controls_source_freshness_calibration_and_override_rejection - name: Check owned Python files run: | .venv/bin/ruff check openpilot/starpilot/aol/temporary_steering.py openpilot/starpilot/aol/tests/test_hkg_angle_eps_retention.py openpilot/starpilot/aol/transport_pause.py openpilot/system/hardware/hardwared.py openpilot/system/hardware/hardware_timing.py openpilot/system/hardware/tests/test_hardware_timing.py openpilot/starpilot/ui/tests/test_weather.py diff --git a/openpilot/starpilot/flm/tests/test_pid_global_capture.py b/openpilot/starpilot/flm/tests/test_pid_global_capture.py index c6c42de71b..9839f7db79 100644 --- a/openpilot/starpilot/flm/tests/test_pid_global_capture.py +++ b/openpilot/starpilot/flm/tests/test_pid_global_capture.py @@ -41,7 +41,7 @@ def test_actual_controls_pid_collects_own_trace_without_changing_disabled_baseli assert candidate.LaC.ff_factor==baseline.LaC.ff_factor assert candidate.CP.as_builder().to_bytes()==original evidence=runtime.capture_evidence(control_ns=now,state_ns=now,active=True,limited=False) - assert evidence is not None, repr({'reason': runtime.reason, 'eligible': runtime.eligible, 'healthy': runtime.healthy, 'active': runtime.active, 'source': runtime.source, 'frame': runtime.frame, 'observation': vars(runtime.observation), 'identity': runtime.identity}) + assert evidence is not None assert evidence.stage_schema_revision==REVISION stage=dict(evidence.stage_values) assert 'friction' not in stage and 'effective_factor' not in stage diff --git a/openpilot/starpilot/flm/tests/test_registered_controls_global.py b/openpilot/starpilot/flm/tests/test_registered_controls_global.py index 16537495e5..d0d3212713 100644 --- a/openpilot/starpilot/flm/tests/test_registered_controls_global.py +++ b/openpilot/starpilot/flm/tests/test_registered_controls_global.py @@ -200,6 +200,7 @@ def controls_for(cp, mode=None, enabled=False): mock.patch('openpilot.selfdrive.controls.controlsd.messaging.PubMaster'), ): params = Params() + params.put_bool('OpenpilotEnabledToggle', True, block=True) selected = published_cp(cp, params) params.put('CarParams', selected.to_bytes(), block=True) params.put(KEY, replace_preference(None, cp, enabled), block=True)