home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.81k stars 30.51k forks source link

Configure LZW36 fan with correct parameter for "breeze" mode #51056

Closed mew1033 closed 2 years ago

mew1033 commented 3 years ago

The problem

The Inovelli LZW36 light and fan controller has at least one missing feature in the current HA entity implementation. If you set the speed to 1, it will enable "breeze" mode. Since HA is already treating this device special here https://github.com/home-assistant/core/blob/dev/homeassistant/components/zwave_js/discovery.py#L206-L218, I was hoping there was a way to also include the "breeze" mode.

Here is the current state from the developer tools:

speed_list:
  - 'off'
  - low
  - medium
  - high
preset_modes: []
speed: medium
percentage: 66
percentage_step: 1.0101010101010102
preset_mode: null
friendly_name: My Fan (Device)
supported_features: 1

What is version of Home Assistant Core has the issue?

2021.5.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

No response

Link to integration documentation on our website

No response

Example YAML snippet

speed_list:
  - 'off'
  - low
  - medium
  - high
preset_modes: []
speed: medium
percentage: 66
percentage_step: 1.0101010101010102
preset_mode: null
friendly_name: Kids Room Fan (Device)
supported_features: 1

Anything in the logs that might be useful for us?

No response

Additional information

No response

probot-home-assistant[bot] commented 3 years ago

Hey there @home-assistant/z-wave, mind taking a look at this issue as its been labeled with an integration (zwave_js) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

github-actions[bot] commented 3 years ago

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 has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

MartinHjelmare commented 3 years ago

Please create a dump of your Z-Wave network and upload the dump as a text file here.

The dump tool is available in the configuration panel of the Z-Wave JS integration, reached from the integrations page in the Home Assistant GUI.

The network dump will help us troubleshoot your device.

Thanks!

mew1033 commented 3 years ago

@MartinHjelmare here's the uploaded network dump: zwave_js_dump.json.txt

I also came up with a method to solve this with fan templates, but the PR was closed. https://github.com/home-assistant/core/pull/51607

Let me know what you think.

raman325 commented 3 years ago

There is nothing in the dump that references a breeze mode, nor is it in the manual: https://support.inovelli.com/portal/api/kbArticles/407409000002102001/attachments/kw8eq50c812f5d3344bc08295860b8b394c7c/content?portalId=edbsn8bb12d5f5a97d693e61ce4ce8a23f8b0e323eac44a6c2794f893e912acf005df&inline=true

Can you share an official reference to this feature? It seems to be a custom thing that is being done out of spec, so I'm not sure if we can accommodate it.

kpine commented 3 years ago

The official Inovelli device drivers implement it, for example Hubitat maps it to an "auto" fan speed.

It is definitely product specific.

mew1033 commented 3 years ago

@kpine is correct. Basically Inovelli wanted to add in a "breeze" mode that randomly cycles to different fan speeds. They did it by changing their firmware so that when the switch is set to 1%, breeze mode is active.

raman325 commented 3 years ago

right, we will have to think about how to implement this in a way that it's reusable, as we are not creating new device specific entity classes

github-actions[bot] commented 2 years ago

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 has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.