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

Enable transitions unexpected behivior #4446

Open
1 task done
dosipod opened this issue Dec 31, 2024 · 4 comments
Open
1 task done

Enable transitions unexpected behivior #4446

dosipod opened this issue Dec 31, 2024 · 4 comments
Labels
bug confirmed The bug is reproducable and confirmed

Comments

@dosipod
Copy link
Contributor

dosipod commented Dec 31, 2024

What happened?

Enable transitions and you will see leds speed up when changing brightness or other things , testing with DDP
Enable_transitions2

To Reproduce Bug

Enable transitions

Expected Behavior

No sudden speed up

Install Method

Self-Compiled

What version of WLED?

WLED 0.16.0-dev (build 2412040)

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@dosipod dosipod added the bug label Dec 31, 2024
@DedeHai
Copy link
Collaborator

DedeHai commented Dec 31, 2024

That is by design, during transitions, FPS limits are disabled. Not that I think its a good design but there appears to be some reason behind it. Maybe it is outdated.
@softhack007 is this behaviour still needed after you updated FPS limit handling?

@softhack007
Copy link
Collaborator

softhack007 commented Dec 31, 2024

@softhack007 is this behaviour still needed after you updated FPS limit handling?

🤔 good point. Actually it's not necessary. I think the reason for this speedup was the "solid" effect - it normally runs at 3fps so colors were flickering during brightness or color change. It might be better to change the effect so it does return FRAMTIME during transitions, and remove the speedup code.

@softhack007 softhack007 added the confirmed The bug is reproducable and confirmed label Dec 31, 2024
@blazoncek
Copy link
Collaborator

blazoncek commented Jan 2, 2025

FYI any effect that is not returning FRAMETIME will exhibit speedup/slowdown. That is by design of the particular effect and the fact that during transition (if you want it to look smooth) strip has to update frequently.

@blazoncek
Copy link
Collaborator

ICU is another example.

softhack007 added a commit that referenced this issue Jan 10, 2025
this still raises FPS, but only for effects that normally run very slow (solid).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug confirmed The bug is reproducable and confirmed
Projects
None yet
Development

No branches or pull requests

4 participants