-
Notifications
You must be signed in to change notification settings - Fork 420
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
l3-5m-p, brightness command different from ewlink #1517
Comments
You needs to collect debug logs for your cases |
This logs from some TH device. I thought we talking about bulb. |
Apologies... filterd for the correct device.
|
|
I did some more testing and got debug logs from when I control it from ewlink and from HA. Controlling from Ewelink, changing colour and brightness results in sending both colour and brightness for either a brightness update or colour update.
When I control from HA I get either brightness or colour information per update, seems like function def get_params on line 361 controls that. updating brightness results in: or updating colour results in: Perhaps that if is not correct and both values need to be sent? |
First of all, great code, much appreciated. I've tried to read where the issue is but am missing some insights on inputs and outputs. I assume it's some default value assigned to brightness.
Scenario,
pick green 100% brightness
Using brightness slider in Lovelace reduce brightnes to 30%
Lovelace UI shows green 30%
Expected result green color, brightness 30%
Actual output of the strip a blue color, brightness 30% ?
This is true for all colors,
it will always default to blue and then adjust brightness but not retain the original color as you would expect and what the ui shows.
When I change color in ha ui, but keep the brightness the same in ha ui. The color is updated in ewlink app, but the brightness is set to 100%, the light strip goed to 100% also.
When using ewlink the ui displays the correct brightness and color in the strip and in the home assistant ui. So the issue is from state switch originating from home assistant to the ledstrip.
No errors found in the logs as there is no error
L3-5M-P by SONOFF Firmware: 1000.2.1060
The text was updated successfully, but these errors were encountered: