Skip to content

Commit 96ff0e4

Browse files
committed
refactor(update_models_dropdown): Added guard clause removed print
1 parent 0983557 commit 96ff0e4

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

src/FreeScribe.client/UI/SettingsWindow.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -528,16 +528,16 @@ def update_models_dropdown(self, dropdown, endpoint=None):
528528
the dropdown widget in the settings window with the new list of models.
529529
"""
530530
if self.editable_settings_entries[SettingsKeys.LOCAL_LLM.value].get():
531-
print("Using local LLM, skipping model update")
531+
return
532+
533+
dropdown["values"] = ["Loading models...", "Custom"]
534+
dropdown.set("Loading models...")
535+
models = self.get_available_models(endpoint=endpoint)
536+
dropdown["values"] = models
537+
if self.editable_settings[SettingsKeys.LOCAL_LLM_MODEL.value] in models:
538+
dropdown.set(self.editable_settings[SettingsKeys.LOCAL_LLM_MODEL.value])
532539
else:
533-
dropdown["values"] = ["Loading models...", "Custom"]
534-
dropdown.set("Loading models...")
535-
models = self.get_available_models(endpoint=endpoint)
536-
dropdown["values"] = models
537-
if self.editable_settings[SettingsKeys.LOCAL_LLM_MODEL.value] in models:
538-
dropdown.set(self.editable_settings[SettingsKeys.LOCAL_LLM_MODEL.value])
539-
else:
540-
dropdown.set(models[0])
540+
dropdown.set(models[0])
541541

542542
def set_main_window(self, window):
543543
"""

0 commit comments

Comments
 (0)