Skip to content
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

Adds description on how to use intent slots #2638

Closed
wants to merge 1 commit into from

Conversation

HarvsG
Copy link

@HarvsG HarvsG commented Jul 16, 2024

Partially fixes #2636 for me

Likely needs some indentation

@HarvsG
Copy link
Author

HarvsG commented Jul 16, 2024

This works even better, but it is probably a bit of a reach.

description = f"Play media on a media player, `{QUERY_SLOT}:` sends a query to an AI conversation agent if configured for ambiguous commands and should not be used by AI agents, `{TRACK_SLOT}:`, `{ARTIST_SLOT}:`and `{ALBUM_SLOT}:` should be preferred. `{NAME_SLOT}:` should not include `_`."

@OzGav OzGav requested a review from jozefKruszynski July 16, 2024 21:40
@tronikos
Copy link
Contributor

This should be closed. Proper fix is in #2685

@OzGav OzGav closed this Jul 27, 2024
@HarvsG HarvsG deleted the patch-1 branch July 27, 2024 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AI is confused by the query ambiguity in the intent
3 participants