Aircoookie / WLED

Control WS2812B and many more types of digital RGB LEDs with an ESP8266 or ESP32 over WiFi!
https://kno.wled.ge
MIT License
15.01k stars 3.24k forks source link

FX improvements and cleanup #4145

Open DedeHai opened 2 months ago

DedeHai commented 2 months ago

This has been on my todo list for a long time, finally done. It adds some new functionality to 2D FX (blurring / smearing / palette support) and saves on code size (merged 1D FX) Note: Polar Lights is now slightly different as no palette matches the original effect palette

blazoncek commented 2 months ago

@ewoudwijma @netmindz would you care to take a look as some effects were created by you.

  • Changed default of Palette effect: Animate rotation is now unchecked by default (it looks weird and confusing in 1D as a default)

This will need to be checked with @triplewhy

DedeHai commented 2 months ago

I did not consider backward compatibility to be so important but you are right, will revert default behaviour back to what it was before. edit: should meteor_smooth be kept as an alias or is removing it ok?

blazoncek commented 2 months ago

should meteor_smooth be kept as an alias or is removing it ok

As we removed some effects in 0.13 and then again in 0.14 I think we can do that again in favour of the possibility to finally merge in PS effects. I would really like to see those (at least 2D) in.

netmindz commented 2 months ago

@ewoudwijma @netmindz would you care to take a look as some effects were created by you.

  • Changed default of Palette effect: Animate rotation is now unchecked by default (it looks weird and confusing in 1D as a default)

This will need to be checked with @TripleWhy

I don't think any of those were added by me, but I can try and take a look at this PR in the next few days

DedeHai commented 1 month ago

last two commits (pal0 override and palette gradient for scrolling text) are just suggestions for review/comment.

DedeHai commented 1 month ago

This is now a little better but still does not cover HTTP API.

a little hint which function handles this?

blazoncek commented 1 month ago

a little hint which function handles this?

set.cpp ; handleSet()

DedeHai commented 1 month ago

handleSet() calls seg.setMode() which takes care of setting the default palette. what am I missing?

blazoncek commented 1 month ago

what am I missing?

Then you are all set. I do not know every line of code from the top of my head, but I do now a few relations that may be hidden.

blazoncek commented 1 month ago

Ah, disregard. 🤦

DedeHai commented 1 month ago

maybe cherry-pick the latest commit before 0.15 release?(https://github.com/Aircoookie/WLED/pull/4145/commits/6a269447f6438e53835545b51d4ce2e3ebee5b15 )

blazoncek commented 1 month ago

maybe cherry-pick the latest commit before 0.15 release?(6a26944 )

As this is a bugfix you can edit and push 0_15 branch directly. Just don't do it like you did for #3877 😄 use AC repo.