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

Dissolve/Dissolve Rnd with secondary color stopped working in 0.14 #3502

Closed
1 task done
TripleWhy opened this issue Nov 5, 2023 · 7 comments
Closed
1 task done
Labels
bug effect fixed in source This issue is unsolved in the latest release but fixed in master

Comments

@TripleWhy
Copy link

TripleWhy commented Nov 5, 2023

What happened?

Dissolve doesn't do much when there is a background color set, that isn't black:
It will dissolve to to the background color, and keep all LEDs at the background color indefinitely.

To Reproduce Bug

  1. Use WLED 0.14.
  2. Select the Dissolve (or Dissolve Rnd) effect.
  3. Chose a background color that isn't black.

Expected Behavior

Fills LEDs with primary in random order, then with background color, then repeats.

Install Method

Binary from WLED.me

What version of WLED?

WLED 0.14.0 (build 2310130)

Which microcontroller/board are you seeing the problem on?

ESP8266

Relevant log/trace output

No response

Anything else?

This worked in WLED 0.13.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@TripleWhy TripleWhy added the bug label Nov 5, 2023
@blazoncek
Copy link
Collaborator

It works as expected on my system.

@blazoncek blazoncek added the cannot reproduce Developers are not able reproduce. Might be fixed already, or report is missing important details label Nov 5, 2023
@TripleWhy
Copy link
Author

TripleWhy commented Nov 6, 2023

I played around some more, and it currently seems to me that:

  • The issue doesn't exist on the ESP32, just the ESP8266.
  • The issue only happens when the overall brightness is < 200.

@blazoncek
Copy link
Collaborator

Nope. Works on ESP32, ESP32-S2, SEP32-C3 and ESP8266.
Matrix or not. Brightness 200 or less, palette Default or any other. Fx and Bg any color. Random or not.

Must be something particular to your set-up. Have you tried "Global LED buffer"?

@TripleWhy
Copy link
Author

Enabling that option does indeed fix this.

When I copy a configuration backup from my ESP8266 to my ESP32, I have to enable the option as well, otherwise I get the same issue.
However, when I erase the ESP32, re-flash the firmware, and try with a clean setup, I don't need the option. Strange, but I don't know if you consider that an issue or not.

@blazoncek
Copy link
Collaborator

So the problem is lossy getPixelColor(). Good to know.

@blazoncek blazoncek added effect and removed cannot reproduce Developers are not able reproduce. Might be fixed already, or report is missing important details labels Nov 7, 2023
@blazoncek
Copy link
Collaborator

The problem lies in this comparison: SEGMENT.getPixelColor(i) == SEGCOLOR(1) which may never be true if the global brightness (or even segment opacity) is not 255.

@blazoncek
Copy link
Collaborator

I think I have it sorted out.

blazoncek added a commit that referenced this issue Nov 8, 2023
blazoncek added a commit that referenced this issue Nov 8, 2023
@softhack007 softhack007 added the fixed in source This issue is unsolved in the latest release but fixed in master label Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug effect fixed in source This issue is unsolved in the latest release but fixed in master
Projects
None yet
Development

No branches or pull requests

3 participants