Merge remote-tracking branch 'openms/main'

Author: t0mdavid-m

src/workflow/ParameterManager.py

@@ -62,6 +62,9 @@ def save_parameters(self) -> None:
             # get all session state param keys and values for this tool
             for key, value in st.session_state.items():
                 if key.startswith(f"{self.topp_param_prefix}{tool}:1:"):
+                    # Skip display keys used by multiselect widgets
+                    if key.endswith("_display"):
+                        continue
                     # get ini_key
                     ini_key = key.replace(self.topp_param_prefix, "").encode()
                     # get ini (default) value by ini_key

src/workflow/StreamlitUI.py

@@ -869,13 +869,25 @@ def display_TOPP_params(params: dict, num_cols):
                         if len(p['valid_strings']) > 0:
                             # Filter current values to only include valid options
                             current_values = [v for v in p["value"] if v in p['valid_strings']]
+
+                            # Use a display key for multiselect (stores list), sync to main key (stores string)
+                            display_key = f"{key}_display"
+
+                            def on_multiselect_change(dk=display_key, tk=key):
+                                st.session_state[tk] = "\n".join(st.session_state[dk])
+
                             cols[i].multiselect(
                                 name,
                                 options=sorted(p['valid_strings']),
                                 default=current_values,
                                 help=p["description"],
-                                key=key,
+                                key=display_key,
+                                on_change=on_multiselect_change,
                             )
+
+                            # Ensure main key has string value for ParameterManager
+                            if key not in st.session_state:
+                                st.session_state[key] = "\n".join(current_values)
                         else:
                             # Fall back to text_area for freeform list input
                             cols[i].text_area(