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.04k stars 3.25k forks source link

fixed palette FX to more closely match original 1D version #4263

Open DedeHai opened 1 week ago

DedeHai commented 1 week ago

We frequently get confused beta users because rotation is enabled by default and sliders need to be fiddled with to arbitrary but exact positions to get an approximate match of the original 1D palette effect. Also it was not possible to make it match exactly: the moving direction was inverted. The shifted rotation angle now makes it possible to choose the direction. This does not fix everything but tries to match it as closely as possible, an exact match is not possible unfortunately. Existing presets will match worse with this update, but at least it is possible to make them match again. It is basically a question of which version will annoy more users: having an approximate match on old presets but being unable to match it exactly or have a complete missmatch but the possibility to set it the same again. Another downside: presets made in any 0.15 beta will also not match anymore...

with the new default values, selecting the palette effect will give a close match to the original in 1D but will be different than the 2D beta version up to this point.

If this PR is to be merged it better be done so before 0.15 release. The minimum that should be changed are the default check settings.

DedeHai commented 1 week ago

There was some initial discussion about this in the original PR: https://github.com/Aircoookie/WLED/pull/3683

softhack007 commented 1 week ago

If this PR is to be merged it better be done so before 0.15 release. The minimum that should be changed are the default check settings.

I agree with your reasoning 👍 should be for 0.15.0, to avoid breaking previous behaviour.

DedeHai commented 1 week ago

@TripleWhy @dosipod can you please check if this looks ok to you and if its the desired improvement?