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

KNX Climate and Alexa #30204

Closed
sagitt opened this issue Dec 24, 2019 · 18 comments
Closed

KNX Climate and Alexa #30204

sagitt opened this issue Dec 24, 2019 · 18 comments

Comments

@sagitt
Copy link

sagitt commented Dec 24, 2019

Home Assistant release with the issue:
0.103.4

Last working Home Assistant release (if known):
n/a

Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io

Integration:
https://www.home-assistant.io/integrations/climate.knx/

Description of problem:
With HomeKit all works fine, but with alexa, if configured as below won't work.
only works with "on_off_address\state_address" or with "controller_mode_address\mode_state_address" configured.

This happen because Alexa NEED the "hot" or "cold" mode. My thermostat only have to set the temperature, not have a manual on\off or controller_mode, when it feel the right temperature activate valves.

If i set the on\off, i'm not able to set te temparature without before start it, and for start it i need to set the temperature.

For now, i configured the controller_mode_address with (for now not used) daikin knx module, and all works fine. I'm using it with a knx\ip interface, but i have "Thinka" gateway too and it ask if the thermostat is heating only or not and not need any controller mode.

I think the climate knx integration need a virtual setup for this thing or fix it for alexa.

thanks

HomeKit

  • platform: knx
    name: 'Riscaldamento [Zona Giorno] HomeKit'
    temperature_address: '3/0/0'
    target_temperature_address: '3/1/0'
    target_temperature_state_address: '3/2/0'
    min_temp: 7.0
    max_temp: 35.0

Alexa

  • platform: knx
    name: 'Riscaldamento [Zona Giorno] Alexa'
    temperature_address: '3/0/0'
    target_temperature_address: '3/1/0'
    target_temperature_state_address: '3/2/0'
    controller_mode_address: '6/1/2'
    controller_mode_state_address: '6/1/3'
    min_temp: 7.0
    max_temp: 35.0

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):


Additional information:

@probot-home-assistant
Copy link

Hey there @home-assistant/cloud, @ochlocracy, mind taking a look at this issue as its been labeled with a integration (alexa) you are listed as a codeowner for? Thanks!

@ochlocracy
Copy link
Member

Can you provide the following attributes for your climate entity.

hvac_modes:
preset_modes:
supported_features:

@sagitt
Copy link
Author

sagitt commented Dec 25, 2019

See this:

Test is without on/off

Now i have problems with homekit too. For example thinka gateway only show temperature set. I’m now using this only for this.

4D15E9A1-B8B0-4873-BA58-1ACA12A64CF1
81719F70-1450-4123-9B41-C34E5DC10C02
5C07A1FF-1A24-4768-9DF8-E0874BA4CF07
85B38BD1-F877-4338-8D9A-49DCCEE0561D

@ochlocracy
Copy link
Member

https://github.com/home-assistant/home-assistant/blob/b4d6d238e5647a74d9bd7f4dd8b6810a178d1d8f/homeassistant/components/alexa/capabilities.py#L902-L917

For situations like this where hvac_modes is None. It may be possible to report thermostatMode as AUTO or the undocumented CUSTOM mode instead of raising the UnsupportedProperty. I will test the implications of this against the Alexa API.

Otherwise this may need to be handled in the KNX integration.

@pvizeli
Copy link
Member

pvizeli commented Jan 8, 2020

The issue is the KNX handles the climate model wrong, and there is no maintainer on KNX; they can fix the problem. KNX is currently unmaintained. We need not fix alexa, someone need fix the KNX.

@probot-home-assistant
Copy link

Hey there @Julius2342, mind taking a look at this issue as its been labeled with a integration (knx) you are listed as a codeowner for? Thanks!

@Julius2342
Copy link
Contributor

Julius2342 commented Jan 8, 2020

KNX has an own understanding of climate modes (and within different KNX climate modules it is terribly complicated already to have a good abstraction, bc the devices of every manufacturer handle these modes slightly different.).

There was huge trouble when HASS introduced the new UI and the different climate modes were handled less dynamic compared to the way before.

@Julius2342
Copy link
Contributor

@sagitt : as KNX does (as far as i know) not have any hot or cold mode defined, what would be the expected climate mode, the device should be set to after saying: "make room x hot" ?

@sagitt
Copy link
Author

sagitt commented Jan 9, 2020

@sagitt : as KNX does (as far as i know) not have any hot or cold mode defined, what would be the expected climate mode, the device should be set to after saying: "make room x hot" ?

Hi, my knx climate only set the temperature and activate valves (floor heating)

This is what i see with my gateway thinka. Now i’m using it only for this.

Biticino LN4691KNX is the model

D424CA2C-D59A-4F8D-B784-ACBA6F04FC8F

E4B9DA09-C332-4C45-A3DE-BD8BE52FD312

16BECE8A-3395-4D8E-89CE-C9A27072681D

@Julius2342
Copy link
Contributor

@pvizeli : we would need some help, what the expected behaviour of a component should be, if there is no heat/cool mode supported by the device but other components (e.g. Alexa) expect such functionality. Wouldn't this be something which should be solved within either the base component or Alexa itself?

@stale
Copy link

stale bot commented Apr 8, 2020

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 8, 2020
@sagitt
Copy link
Author

sagitt commented Apr 15, 2020

i tested it with new releases. same problem

@stale stale bot removed the stale label Apr 15, 2020
@balloob
Copy link
Member

balloob commented Apr 22, 2020

@Julius2342 the climate entity should return heat as current mode and [heat] for available hvac modes.

@Julius2342
Copy link
Contributor

@balloob : the knx/climate entity has some advanced logic for hvac-modes. Would it fix the problem, if the entity defaults to HEAT if nothing is configured?

@sagitt
Copy link
Author

sagitt commented Apr 29, 2020

I tested it with 0.109, same issue. I don’t know if there is a specific configuration for this type. Anyway my thinka knx gateway still works in right way and don’t need on/off but only temperature setpoint.

Thinka:
987D162A-74C1-418C-8458-3F306296EF5D
313BA0E2-A34D-4EAA-93D6-76C83665AC02
B6435C43-8E07-4909-B334-5A1853985190
4FA25E86-3650-4AAA-8F8E-AD818437D3AA

Hass:
A033123A-5EAB-4E79-9037-E18D656EF8A3
1AFA56C9-0963-436A-8784-55D3A47D5043
5DEE8CCB-1D27-4246-80C0-BA7AB238050A

@balloob
Copy link
Member

balloob commented Apr 29, 2020

@Julius2342 yes. You should always return an HVAC mode. It's mandatory.

@farmio
Copy link
Contributor

farmio commented May 28, 2020

@sagitt
Is this issue resolved now with 0.110?

@sagitt
Copy link
Author

sagitt commented May 28, 2020

@sagitt
Is this issue resolved now with 0.110?

TESTED NOW!!!
Finally WORKS

See "TEST2" items, "TEST1" have on\off and not works.

thanks!

Schermata 2020-05-28 alle 09 01 21

IMG_4888
IMG_4887

IMG_4889
IMG_4890

@balloob balloob closed this as completed May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants