-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Allow changing LLM Task preferences #25779
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
0cca3ba
to
464a77c
Compare
806a58c
to
0d6f768
Compare
@@ -53,6 +54,12 @@ export class HaConfigVoiceAssistantsAssistants extends LitElement { | |||
></assist-pref> | |||
` | |||
: nothing} | |||
${isComponentLoaded(this.hass, "ai_task") | |||
? html`<ai-task-pref |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if this should be on the voice assistants settings page, as it is not related to voice assistants but for tasks in the UI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's LLM related, which I guess sits "above" voice assistants. I don't really know the best place right now so just went for here. Would you prefer a different page ? It's like 1 dropdown right now…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is ok to merge for now I would say, I will ask UX for advice
Breaking change
Proposed change
Allow setting preferred entities for LLM Tasks.
Requires home-assistant/core#146734Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: