-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add Original Broadlink PR#336 #33
Comments
Closes #33 Also added check for "white" hex data
Hi @mitch7391 I've added the white code (with one tweak). Could you test by re-installing this plugin using this command (to put the branch direct from git): If you're able to add the white hex code to your config.json and confirm everything works as expected I'll merge this with the master. |
@kiwi-cam thanks for giving this a go. I just tried to test it a heap and it worked maybe once or twice by turning white, but not when I would expect it to. After selecting a colour from the colour wheel, I would drop the saturation to the center of the wheel (original PR required under 10% saturation from memory), it would either change to the next closest colour or not change at all. The few times it changed to white were when I was expecting a colour (at high saturation). I am not sure if the fact that I have only configured 5 colours plus white out of my total 16 is affecting it; as the code tries to pick the nearest one configured in the wheel. |
Thanks for the update @mitch7391. I've added some logging to show the hue and saturation it's aiming for, as well as what it's setting. If you're up for it, could you run the command again to install over the top and post your logs showing the hue changes? |
@kiwi-cam sorry for the delay, work has been crazy! Ok so I updated and tried again. The order of hues from the colour wheel that I executed were as follows (each at high saturation), started already on white, selected red, selected green, selected white (center of wheel) and selected red again. What I saw: No change from white, change to green, change to red and change to white. Logs: Oct 27 08:15:12 homebridge homebridge[6641]: [10/27/2019, 8:15:12 AM] [Broadlink RM] LightTest setHue: 1 Oct 27 08:15:21 homebridge homebridge[6641]: [10/27/2019, 8:15:21 AM] [Broadlink RM] LightTest setHue: 99 Oct 27 08:15:25 homebridge homebridge[6641]: [10/27/2019, 8:15:25 AM] [Broadlink RM] LightTest setHue: 0 Oct 27 08:15:30 homebridge homebridge[6641]: [10/27/2019, 8:15:30 AM] [Broadlink RM] LightTest setHue: 7 |
Thanks for that @mitch7391. It looks to me like the Saturation value is being set after the Hue. The lines "setSaturation: xx" at the end of the steps appear to match correctly with the next change picking up white. At the moment the setSaturation function isn't being used so I've moved the logic around "white" there - which makes more sense anyway. Could you run |
@kiwi-cam just letting you know I haven't forgotten about this one, I just haven't had much time lately to give it a go. Hopefully should be able to do some further testing for in the next few days. |
No pressure @mitch7391 but any updates? Just having a tidy up and I've for a Pull Request all lined up ready to go when/if you give the all clear. |
@kiwi-cam I am so sorry for the delay in this, have been hands off homekit for a few weeks. I downloaded the link you provided and gave it a go. Order of hues I attempted (started on white): Red, White, Green, White, Blue and then White. What I saw: No change (from white) Red, Green, Red, Blue and then Red. Logs: Nov 26 19:58:59 homebridge homebridge[14205]: [11/26/2019, 7:58:59 PM] [Broadlink RM] LightTest getHue: 0 |
Hope this helps! |
Thanks @mitch7391. Sorry, I'm a little confused trying to line up the colour changes with your attempts:
It's like it missed the first White change, then the others went Red instead?? In the logs things look better. You set white: Is your white hexcode definitely correct... not Red? |
Don’t worry @kiwi-cam I am very confused too because the results I get are inconsistent at times, sometimes it makes no change (like it missed the change altogether). But yes the whites I tried to set that time between each hue set it to red each time instead of white (like it latched the colour I first attempted but it missed); tonight when I get home from work, I will re-learn my red and white codes to rule this out as an issue. However, I have not had an issue with the colours when I have each colour as a simple switch. |
@kiwi-cam i have double checked my learned codes and tested again in the same order and got the similar results. It is really weird but sometimes it seems it has a hard time differentiating between red and white (and their codes are completely different). Sometimes white would execute when I wanted it to, other times it would turn red. Sometimes when setting saturation to low from blue or green, it would turn red for a second and then turn white, but it was not always consistent with repeated tests. |
I know you said you only support the TV accessory on this fork, but I saw you were willing to add some air con mods someone put forward. Frustratingly the old plugin is no longer supported and a PR on it I need was never merged (lprhodes#336) is there any chance you could merge it into your fork? There are two parts to it, I’m only after the white colour added to the light.
The text was updated successfully, but these errors were encountered: