Fix planner_llm_model_name Not Loaded Correctly When Using "Load Config"

[knowlet]

Solution

• In the WebuiManager.load_config method, retrieve the value of planner_llm_provider before setting the model name.
• When setting the value for planner_llm_model_name, dynamically update its choices based on the provider, and use gr.Dropdown(value=..., choices=..., allow_custom_value=True) to update the component.
• All other components remain unchanged.

Impact

• When using the "Load Config" feature, the planner_llm_model_name dropdown will now correctly display the model name according to the loaded configuration, without requiring manual reselection.
• This approach can be extended to other dropdowns with similar dependencies if needed.

Testing

1. In the UI, select different values for planner_llm_provider and planner_llm_model_name, then save the configuration.
2. Restart the WebUI or switch configuration files, and use "Load Config" to load the saved configuration.
3. Verify that planner_llm_model_name correctly displays the corresponding model name, and that the provider and model name are properly synchronized.

Screenshot

Reference

fix #589

---

Summary by mrge

Fixed the "Load Config" feature so the planner LLM model name dropdown now shows the correct model based on the selected provider.

• Bug Fixes
  • The dropdown updates its choices and value automatically when loading a config, matching the saved provider and model name.

[CLAassistant]

All committers have signed the CLA.

[cubic-dev-ai]

mrge reviewed 1 file and found no issues. Review PR in mrge.io.

[vvincent1234]

planner_llm_model_name should set to None because we don't want llm_panner in default

[knowlet]

Change the default back to "None" and fix it by yielding an update first.

[vvincent1234]

Thank you. You don't need a async , just use import time and time.sleep(0.2), i will let this pr pass

[knowlet]

You're right, I've changed back to not using async and using time.Sleep.

time.Sleep didn't work the first time I tested it because I forgot to yield update_components. I switched to async while testing other methods and forgot to switch back. Thanks for the reminder!