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
73.54k stars 30.72k forks source link

Vesync fan speed #40552

Closed alexkirkp closed 4 years ago

alexkirkp commented 4 years ago

The problem

The new vesync integration for the Levoit fan LV-PUR131 isn't behaving as expected with regards to fan speed, the home assistant settings don't match up with the actual fan settings.

HA Setting Fan action
auto auto
off fan goes to low and turns off
low medium
medium high
high nothing

Environment

Problem-relevant configuration.yaml

This integration was setup with the UI.

Traceback/Error logs

Additional information

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

Hey there @markperdue, @webdjoe, @thegardenmonkey, mind taking a look at this issue as its been labeled with an integration (vesync) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

TheGardenMonkey commented 4 years ago

Looks like the fan_speeds array needs to be corrected to have AUTO at the end. Doing some testing now and will submit a pull request for it.

Line 25 of the fan.py file... FAN_SPEEDS = [SPEED_OFF, SPEED_LOW, SPEED_MEDIUM, SPEED_HIGH, SPEED_AUTO]