From 40e0675fe6ad829c139399548a3c69a1ad95bb71 Mon Sep 17 00:00:00 2001 From: Jason Wen Date: Wed, 26 Aug 2026 02:49:37 -0400 Subject: [PATCH] ui: record why model names read the params slots and not modelManagerSP --- openpilot/selfdrive/ui/sunnypilot/model_info.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/openpilot/selfdrive/ui/sunnypilot/model_info.py b/openpilot/selfdrive/ui/sunnypilot/model_info.py index ce0d1cdf79..2826a19f13 100644 --- a/openpilot/selfdrive/ui/sunnypilot/model_info.py +++ b/openpilot/selfdrive/ui/sunnypilot/model_info.py @@ -23,7 +23,11 @@ def bundles_for_source(source: str): def model_info() -> tuple[str, str, str]: - """returns (active source, active model name, other model name)""" + """returns (active source, active model name, other model name) + + Names come from the params slots, never modelManagerSP.activeBundle — the + manager republishes a tick after a chestnut change, so the stale bundle + would flash the wrong model.""" source = active_source() other = "qcom" if source == "usbgpu" else "usbgpu" active_bundle = get_selected_bundle(ui_state.params, source)