Update prompt to add support for playlist voice commands #2833
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adding additional info to the prompt to teach the LLM how to handle requests to play playlists.
|
@jozefKruszynski thoughts? |
|
Looks pretty reasonable. I assume it's been tested by @nikito and works for him with no code changes. |
Yes, tested with a few different playlists of mine. Also tried the original commands like "songs by muse" or individual song requests to ensure no regressions. 🙂 |
marcelveldt
reviewed
Sep 6, 2024
marcelveldt
reviewed
Sep 6, 2024
Co-authored-by: Marcel van der Veldt <[email protected]>
|
Seems to be working fine with the suggested changes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding additional info to the prompt to teach the LLM how to handle requests to play playlists.