Skip to content

Commit

Permalink
Merge pull request #50 from joshuadavidson/fix-correct-assistant-typo…
Browse files Browse the repository at this point in the history
…-throughout

fix: correct assistant typo throughout
  • Loading branch information
TheFes authored Jan 29, 2025
2 parents 41c012e + 0b3c00e commit db266ba
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 9 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ _Bedroom Sophia_ it does not work if you you say _Sophia's Bedroom_ or when the
Speech to Text conversion uses _Sofia_ instead of _Sophia_. Unfortunately aliases
can't be used in the automation, as there is no template to get the aliases of areas
or entities. Capitalization of words does't matter, so _bedroom sophia_ will still
match if the area name in Home Assitant is _Bedroom Sophia_
match if the area name in Home Assistant is _Bedroom Sophia_
* Change the prompt which is used to have the LLM provide the correct data.
You don't need to translate this if you use a different language.

Expand Down Expand Up @@ -187,7 +187,7 @@ The language of the voice command is not relevant, the script has all descriptio
2. Allow the LLM integration to access your house, otherwise it won't be able to use the script as a tool
3. Import the Blueprint using the button above
4. Create a script using the blueprint, optionally you can adjust the prompt settings which are the basis on how the script will be used by the LLM
5. Optionally add additional actions to be performed after the Music Assitant play media step while setting up the script
5. Optionally add additional actions to be performed after the Music Assistant play media step while setting up the script
5. Save the script, make sure to give it a clear description, as that is what will be used by the LLM to determine when to use it (see example below).
6. Expose the script to Assist

Expand Down
6 changes: 5 additions & 1 deletion llm-enhanced-local-assist-blueprint/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

# Changelog: Option 2 - LLM Enhanced automation

## 20250129

* fix: correct assistant typo throughout

## 20250128

* Initial version when the changelog was created
* Initial version when the changelog was created
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ Elke zin moet:

### Hoe het doel voor de media bepaald wordt

1. Indien er één of meerdere areas en/of één of meerdere mediaspelers vermeld wordenin het commando, en deze aan een area en/of speler voor Music Assitant gekoppeld kunenn worden, worden deze gebruikt. Als je de namen niet naar de LLM stuurt, en een area of speler vermeld die niet exact overeen komt, wordt de `No target response` reactie gegeven.
1. Indien er één of meerdere areas en/of één of meerdere mediaspelers vermeld wordenin het commando, en deze aan een area en/of speler voor Music Assistant gekoppeld kunenn worden, worden deze gebruikt. Als je de namen niet naar de LLM stuurt, en een area of speler vermeld die niet exact overeen komt, wordt de `No target response` reactie gegeven.
2. Als er geen area of speler in het commando vermeld wordt, maar er wel een area bepaald kan worden op basis van het apparaat waarvan het commando gestuurd wordt, wordt deze area gebruikt indien er ook een Musuc Assistant mediaspeler in die area is.
3. Indien er geen area of mediaspeler vernoemd wordt, en deze ook niet bepaald kan worden op basis van het apparaat waarvan het commando komt, wordt de media gespeeld op de `Default Player`. Indien die niet ingesteld is, wordt de `No target response` reactie gegeven.

Expand All @@ -67,4 +67,4 @@ Speel liedjes van U2
Speel muziek van de componist van Oppenheimer
Speel dat album met op de voorkant de naakte baby die naar een bankbiljet zwemt
```
'
'
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ blueprint:
the Speech to Text conversion uses _Sofia_ instead of _Sophia_. Unfortunately
aliases can't be used in the automation, as there is no template to get the aliases
of areas or entities. Capitalization of words does't matter, so _bedroom sophia_
will still match if the area name in Home Assitant is _Bedroom Sophia_
will still match if the area name in Home Assistant is _Bedroom Sophia_
* Change the prompt which is used to have the LLM provide the correct data. You
don't need to translate this if you use a different language.
Expand Down
6 changes: 5 additions & 1 deletion llm-script-blueprint/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

# Changelog: Option 3 - Full LLM Script

## 20250129

* fix: correct assistant typo throughout

## 20250128

* Initial version when the changelog was created
* Initial version when the changelog was created
4 changes: 2 additions & 2 deletions llm-script-blueprint/llm_voice_script.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
blueprint:
name: LLM Script for Music Assitant voice requests
name: LLM Script for Music Assistant voice requests
source_url: https://github.com/music-assistant/voice-support/blob/main/llm-script-blueprint/llm_voice_script.yaml
description:
'![Image](https://github.com/music-assistant/voice-support/blob/main/assets/music-assistant.png?raw=true)
Expand Down Expand Up @@ -55,7 +55,7 @@ blueprint:
The variable will undefined in case there was no area information, in case it
is defined it will be a list of area_ids|
|`media_player`|The Music Assitant media player(s) determined by the LLM on which
|`media_player`|The Music Assistant media player(s) determined by the LLM on which
the request will be played. The variable will be undefined in case there was no
media player provided in the request, in case it it defined it should be a list
entity_ids, but it could be the LLM provided names instead.
Expand Down

0 comments on commit db266ba

Please sign in to comment.