home-assistant / intents

Intents to be used with Home Assistant
https://developers.home-assistant.io/docs/voice/overview/
Creative Commons Attribution 4.0 International
452 stars 515 forks source link

[pl] Climate not working #2384

Closed Mis1ify closed 1 month ago

Mis1ify commented 2 months ago

It seems like if I want to set a temperature on a climate, Assist thinks I want to change the light color, not the temperature.

intent: name: HassLightSet slots: {} details: {} targets: {} match: false sentence_template: <set> <name> na {color} [kolor] unmatched_slots: name: 'temperaturę ' color: 30° celsjusza source: builtin

witold-gren commented 1 month ago

Can you add more information how you name your devices and how looks your sentences? If you go to test case in polish language you can find more examples how you can use it. https://github.com/home-assistant/intents/blob/main/tests/pl/climate_HassClimateSetTemperature.yaml

Mis1ify commented 1 month ago

Yes I use this file as an example, and it still doesn't understand me. No matter what sentence from that file I say, it searches for the light, not the climate.

witold-gren commented 1 month ago

I checked it in my instances using Sentences parser and you're right.. in my case they return:

intent:
  name: HassLightSet
slots: {}
details: {}
targets: {}
match: false
sentence_template: <set> <name> na {color} [kolor]
unmatched_slots:
  name: 'temperaturę '
  color: 30 stopni
source: builtin

but it should return HassClimateSetTemperature. I will look on it..

witold-gren commented 1 month ago

I think I found the problem. There is problem with building intents for Home Assistant, because in my configuration there is absolutely nothing related to HassClimateSetTemperature. @Mis1ify you can check your configuration (if you can). In folder /usr/local/lib/python3.12/site-packages/home_assistant_intents/data (you can find this path inside home assistant container: docker exec -it homeassistant /bin/bash) you can find pl.json which contains all intents. In my configuration I don't have intents responsible for set temperature in climate domain. I have already reported it on Discord, but if necessary I will create a ticket. But this is very weird behaviour...

witold-gren commented 1 month ago

https://github.com/home-assistant/intents/blob/main/intents.yaml#L220 This intent is not supported yet directly with HA 😀 that's the reason it's not working for you.

@Mis1ify I think you can close this ticket for now until this configuration is no supported. 😀

Mis1ify commented 1 month ago

Oh okay, thanks for investigation.

witold-gren commented 1 month ago

@Mis1ify If this functionality is important for you you can create custom sentences to set this devices 😀 Polish translation exist so you don't need create everything from the scratch.. there you can find good example how to do it:

  1. https://github.com/xFrieDSpuDx/HomeAssistant-VoiceIntents
  2. https://github.com/tetele/custom_sentences
Mis1ify commented 1 month ago

Thank you!