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

No light with brightness below 24% #36

Open
Sjaakafhaak opened this issue Feb 22, 2022 · 7 comments
Open

No light with brightness below 24% #36

Sjaakafhaak opened this issue Feb 22, 2022 · 7 comments
Labels
bug Something isn't working

Comments

@Sjaakafhaak
Copy link

Sjaakafhaak commented Feb 22, 2022

Hi,

I like your project. But I'm having some issue with the brightness controls.
When I set the brightness below 24% in the Home App the light goes off. So I did some digging around and it seams like that the brightness control in the app triggers both the master and color brightness in WLED.
This results in an accumulation of both and turns off the leds.
A side effect of this is that when I toggle the light with a physical button ( to a macro T=2) the light won't go on because the color brightness is low that the color is black.

So I was wondering if it is possible to fix this behaviour.
I noticed that you are converting between HSV and RGB value I guess because of what homekit is sending out? Maybe the issue comes from that part. I'm no expert at all in coding.

Thanks

--edit
I'm running
homebridge-simple-wled v1.3.3
WLED v0.13.0-b6

@jstrausd
Copy link
Owner

Hello,
I also encountered this problem, i hope i will find time to fix this issue. It is standing at the first place on my list, but I'm currently very very busy.
Thanks for the problem reporting!!

@jstrausd jstrausd added the bug Something isn't working label Feb 25, 2022
@taylorsatula
Copy link

Thank you!

@bvalasek
Copy link

Is there any update on this? I have just found I created duplicate of same issue #48 a week ago. Will close that issue and wait for updates here.

IMG_0882_720_copy.mov

@TonnyCS
Copy link

TonnyCS commented Feb 17, 2023

Hey all, any updates on this one?

@TonnyCS
Copy link

TonnyCS commented Feb 17, 2023

Btw, I dont wanna be "that guy", but from what I can see through the WLED UI, when I change the brightness the only parameters in the json are bri, time and v

From what I can see in the httpSetBrightness method in the wled-accessory.ts the json that is sent is using parameters bri and seg.

I think the seg parameter sets the top slider in WLED.

When I try to simulate the issue, so I use both of the sliders (the top one and the brightness one) in the WLED UI, right around 20% value I get the same issue as described here.

When I set the top slider in the WLED UI, the json has parameters seg, v and time.

I think that the seg parameter is being set incorrectly.

I have no idea how to test Homebrige plugins, so I think I'll leave it up to you @jstrausd. I hope this will help.

@TonnyCS
Copy link

TonnyCS commented Feb 17, 2023

For anyone with the same issue and before the pull request above is completed I created and published a fork of this project with the fix.

You can find it here: https://github.com/TonnyCS/homebridge-simple-wled-brightness-fix.

To use it in HomeBridge just add homebridge-simple-wled-brightness-fix as you normally would with any other plugin and copy and paste your config.json from this plugin.

@drewcovi
Copy link

drewcovi commented Apr 2, 2023

I have no idea how to test Homebrige plugins, so I think I'll leave it up to you @jstrausd. I hope this will help.

It's a matter of firing up a new instance of homebridge, ideally on your development machine, creating a new "test" home and linking to that instance of homebridge with your fixed plug-in. Thanks for this fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants