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

Simulation of Neopixel Ring WS8212 is too dark for low levels #847

Open
georgik opened this issue Jul 22, 2024 · 1 comment
Open

Simulation of Neopixel Ring WS8212 is too dark for low levels #847

georgik opened this issue Jul 22, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@georgik
Copy link

georgik commented Jul 22, 2024

Describe the bug
Simulation of WS8212 smart leds is not visible when levels are "low".

To Reproduce
Set Neopixel Ring color to (1,0,0), it won't be visible in Wokwi Simulation.
If you use single Neopixel, then it's slightly visible.
Here, see the first set of colors, it will be visible on Neopixel, but not on Ring https://wokwi.com/projects/404102270788810753

Expected behavior
Real Neopixel is glowing brightly even when level is set to 1. When set to 255 it's super bright.
Increase the baseline for visible color.

Additional context
The problem was discovered when using Ring for simulation of Swift https://github.com/apple/swift-embedded-examples/blob/main/esp32-led-strip-sdk/main/LedStrip.swift#L40

@sutaburosu
Copy link

I see the RGB value of the pixels in the ring on my screen as (1, 1, 1), so Wokwi is passing through the Neopixel values unchanged.

This doesn't seem to work for rings, but for other addressable LEDs in Wokwi there are additional attributes that influence the mapping of LED values to screen values. matrixBrightness is a multiplication factor, and _temp_gamma is what you desire. See this sketch for an example of gamma at 0.66 on a strip of LEDs.

Issue #240 already covers work to unify all the addressable LED types, which should allow gamma correction to be used with rings too. Please vote it for it to see this work completed sooner.

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

2 participants