From d56329655e8861168a2270b6c0d586ce671fef45 Mon Sep 17 00:00:00 2001 From: edo-neo Date: Thu, 6 Mar 2025 12:09:49 +0100 Subject: [PATCH] Fix handling of PID mode configuration retrieval Updated the PID mode configuration to directly use the combobox object instead of invoking it as a function. Adjusted references accordingly to ensure proper mapping and alignment with the updated usage. --- src/components/tecna_marposs_provaset_t3.py | 2 +- src/ui/leak_step_editor/leak_step_editor.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/components/tecna_marposs_provaset_t3.py b/src/components/tecna_marposs_provaset_t3.py index 8e1ff79..a2bfe78 100644 --- a/src/components/tecna_marposs_provaset_t3.py +++ b/src/components/tecna_marposs_provaset_t3.py @@ -302,7 +302,7 @@ class TecnaMarpossProvasetT3(ModbusComponent): recipe_barcode = f"j{recipe.part_number}"[:16].encode("ascii") recipe_barcode += b"\x00" * (24 - len(recipe_barcode)) test_flags = 0b0110100001010000 if (step.spec.get("autotest", False) in ["ko_check"]) else 0b0110000001010000 - pid_mode_text = step.spec["pid_mod_config"]() # Get the selected text from the combobox + pid_mode_text = step.spec["pid_mod_config"] # Get the selected text from the combobox pid_mode_value = { # Mapping of text to numeric values "AUTO": 5, "FAST": 0, diff --git a/src/ui/leak_step_editor/leak_step_editor.py b/src/ui/leak_step_editor/leak_step_editor.py index 2be9005..8df982d 100644 --- a/src/ui/leak_step_editor/leak_step_editor.py +++ b/src/ui/leak_step_editor/leak_step_editor.py @@ -27,7 +27,7 @@ class Leak_Step_Editor(Editor): "chan_sel": self.chan_sel_sb, "ext_blow_time": self.ext_blow_time_sb, "ext_flush_time": self.ext_flush_time_sb, - #PID + # PID "pid_pressure_correction": self.test_pid_correction_sb, - "pid_mod_config": self.pid_mode_sel.currentText(), + "pid_mod_config": self.pid_mode_sel, })