Berack/upo-app-agents
0
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Fix chat defaults #46

Merged
Berack96 merged 5 commits from fix-chat-default-model into main 2025-10-27 12:41:50 +01:00
Conversation 3 Commits 5 Files Changed 1 +3 -1
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/app/interface/chat.py
View File

@@ -79,6 +79,7 @@ class ChatManager:
with gr.Row(): with gr.Row():
provider = gr.Dropdown( provider = gr.Dropdown(
choices=self.inputs.list_models_names(), choices=self.inputs.list_models_names(),
value=self.inputs.team_leader_model.label,
type="index", type="index",
copilot-pull-request-reviewer[bot] commented 2025-10-24 18:38:23 +02:00 (Migrated from github.com)
Review
Copy Link

Setting value directly on a Gradio component after creation may not update the displayed default. Consider using the value parameter in the gr.Dropdown() constructor instead: value=self.inputs.team_leader_model.label.

Setting `value` directly on a Gradio component after creation may not update the displayed default. Consider using the `value` parameter in the `gr.Dropdown()` constructor instead: `value=self.inputs.team_leader_model.label`.
copilot-pull-request-reviewer[bot] commented 2025-10-24 18:38:23 +02:00 (Migrated from github.com)
Review
Copy Link

Setting value directly on a Gradio component after creation may not update the displayed default. Consider using the value parameter in the gr.Dropdown() constructor instead: value=self.inputs.strategy.label.

Setting `value` directly on a Gradio component after creation may not update the displayed default. Consider using the `value` parameter in the `gr.Dropdown()` constructor instead: `value=self.inputs.strategy.label`.
label="Modello da usare" label="Modello da usare"
) )
@@ -86,6 +87,7 @@ class ChatManager:
style = gr.Dropdown( style = gr.Dropdown(
choices=self.inputs.list_strategies_names(), choices=self.inputs.list_strategies_names(),
value=self.inputs.strategy.label,
type="index", type="index",
label="Stile di investimento" label="Stile di investimento"
) )