Skip to content

Improve error message when default llm is not available - #1736

Merged
igordayen merged 2 commits into
embabel:mainfrom
poutsma:gh-1733
Jun 27, 2026
Merged

Improve error message when default llm is not available#1736
igordayen merged 2 commits into
embabel:mainfrom
poutsma:gh-1733

Conversation

@poutsma

@poutsma poutsma commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This PR improves the error message when the default LLM model is not available, either by suggesting to set embabel.models.default-llm to an LLM that is available, or by suggesting to load models from the Embabel Agent Starter.

Closes gh-1733

This commit improves the error message when the default LLM model is not
available, either by suggesting to set embabel.models.default-llm to an
LLM that is available, or by suggesting to load models from the Embabel
Agent Starter.

Closes embabelgh-1733

Signed-off-by: Arjen Poutsma <poutsma@mac.com>
@poutsma
poutsma requested review from igordayen and johnsonr June 22, 2026 13:04

@igordayen igordayen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks, good, added comment

Updated error message to clarify that models must be loaded.
@igordayen
igordayen merged commit 5c61a70 into embabel:main Jun 27, 2026
13 checks passed
@poutsma

poutsma commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for resolving this, @igordayen

dearshor pushed a commit to dearshor/embabel-agent that referenced this pull request Jun 29, 2026
* Improve error message when default llm is not available

This commit improves the error message when the default LLM model is not
available, either by suggesting to set embabel.models.default-llm to an
LLM that is available, or by suggesting to load models from the Embabel
Agent Starter.

Closes embabelgh-1733

Signed-off-by: Arjen Poutsma <poutsma@mac.com>

* Clarify error message for missing models

Updated error message to clarify that models must be loaded.

---------

Signed-off-by: Arjen Poutsma <poutsma@mac.com>
Co-authored-by: Igor Dayen <165492669+igordayen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants