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

Changing bightness on an effect doesn't change the white channel brightness #1086

Closed
Skaronator opened this issue Mar 14, 2020 · 8 comments
Closed
Labels
bug Something isn't working integration: neopixelbus

Comments

@Skaronator
Copy link
Member

Operating environment/Installation (Hass.io/Docker/pip/etc.):

ESPHome is installed in Docker.
ESPHome 1.14.3
Home Assistant 0.106.6

ESP (ESP32/ESP8266, Board/Sonoff):
ESP32 with SK6812 RGBW Led Strip

Affected component:
light, neopixelbus

Description of problem:

When the light is running a effect, changing the brightness in Home Assistant only take effect for RGB but not for the white channel. Additionally the white value slider does nothing.

As you can see here, when changing the brightness to 0 the white channel continues to stay on.

brightness_100
brightness_0

Not sure what the best option fix for this would be. Either make it that the white value slider works in this case or (even better IMO) apply the brightness for the white channel as well since its all part of the same effect.

Problem-relevant YAML-configuration entries:

light:
  - platform: neopixelbus
    variant: SK6812
    type: grbw
    pin: 26
    num_leds: 144
    name: "${friendly_name}"
    gamma_correct: 1.5
    effects:
      - addressable_lambda:
          name: Debugger
          lambda: |-
            it.all() = ESPColor(255, 0, 0, 50);

Logs (if applicable):

PASTE DEBUG LOG HERE

Additional information and things you've tried:

@farbefreak
Copy link

Seems like its the same bug as : #947

I did try the dev build with the fix, this one handles the color different but still dosed work.

If you set a color in Homeassistant WW/CW turns off completely
like wise If you set WW/CW brightness and choose a RGB color the WW/CW turns off completely
so its impossible to mix RGB with WW/CW

@stale
Copy link

stale bot commented Aug 21, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 21, 2020
@stale
Copy link

stale bot commented Aug 21, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@Skaronator Skaronator removed the stale label Aug 21, 2020
@stale
Copy link

stale bot commented Dec 19, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@probot-esphome
Copy link

neopixelbus source
neopixelbus issues
neopixelbus recent changes
(message by IssueLinks)

@Skaronator Skaronator added bug Something isn't working and removed integration: addressable labels Dec 19, 2020
@jimmycleuren
Copy link

jimmycleuren commented Dec 27, 2020

The same issue happens when using the rest api. Everything is fine when only using the rgb channel and leaving white_value at 0. From the moment you add white_value, the result is always that the white channel is at 100%.

@stale
Copy link

stale bot commented Apr 27, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@oxan
Copy link
Member

oxan commented Jul 10, 2021

This should be fixed with esphome/esphome#1895 merged, available in the upcoming v1.20 release.

@oxan oxan closed this as completed Jul 10, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Nov 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working integration: neopixelbus
Projects
None yet
Development

No branches or pull requests

4 participants