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

Animartrix effects: weird colors in some effects #82

Closed
1 task done
FragMenthor opened this issue Oct 15, 2023 · 15 comments
Closed
1 task done

Animartrix effects: weird colors in some effects #82

FragMenthor opened this issue Oct 15, 2023 · 15 comments
Labels
bug Something isn't working fxed in mdev fixed in latest mdev source code

Comments

@FragMenthor
Copy link

FragMenthor commented Oct 15, 2023

What happened?

Some effects give me colors that do not match the rest of the effect. Nomally is a solid green color that is superimposed on the effect, like some limit has been reached.
Namelly:

  • Complex _Kaleido_3:
    image
  • Hot_Blob:
    image
  • Lava_1:
    image
  • Module_Experiment_2:
    image
  • Module_Experiment_3:
    image
  • Module_Experiment_6:
    image
  • Module_Experiment_7:
    image
  • Module_Experiment_8:
    image
  • Module_Experiment_9:
    image
  • Parametric_Water:
    image
  • SM_10:
    image
  • Water:
    image
  • Zoom_2:
    image

To Reproduce Bug

Don't know if anyone more has this problem, but is shows for me in the UI and it reflects on the LEDs. This shows for me like this since the initial implementation of Animartrix in WLED, and no matter what version I use (_M _S, custom, ...) No change with any version, and I update almost weekly in my test setup with 'mdev'.

Expected Behavior

Smooth and continuous effects, they change abruptly using solid colors, and seem broken somehow.

Install Method

Self-Compiled or other

What version/release of MM WLED?

WLEDMM_0.14.0-b27.32 esp32_4MB_M.bin
build 2310081

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

I don't think this is the expected normal behaviour. If this is intentional, the please close the issue. Thank you!

Code of Conduct

  • I agree to follow this project's Code of Conduct
@FragMenthor FragMenthor added the bug Something isn't working label Oct 15, 2023
@dosipod
Copy link

dosipod commented Oct 15, 2023

The weird colors has been noticed and reported since the initial implementation of Animartrix .as palette is not used we thought that it is still WIP . You have described it better then we did

@FragMenthor
Copy link
Author

FragMenthor commented Oct 15, 2023

What is the possibility of adding pallettes to these effects?

@dosipod
Copy link

dosipod commented Oct 15, 2023

@netmindz could say as he is the one who ported the Animartrix

@netmindz
Copy link
Collaborator

So it doesn't really make sense to use a palette due to the way the effects are designed so I don't see Stefan ever adding that to his code.

There are a few bugs where the colours don't reset properly when you move to certain effects.

Due to the lack of a gamma correction a few don't work in quite the same way as on Smart matrix.

As for blocks of colour with no fade, I've not seen this behaviour

@dosipod
Copy link

dosipod commented Oct 15, 2023

This FX192 Water example , it is really very good as i recall i seen it from Stefan , other then the strange superimposed color which spoils it , thank you for the work though i do hope it could be improved even if as a future plan since you already did the hard part
192_water

@netmindz
Copy link
Collaborator

This fix should certainly help netmindz/animartrix@8fd0df3

Pulled in with #83

@dosipod
Copy link

dosipod commented Oct 17, 2023

I have tried to experiment and change all the values under led preference ,Buffer , Gamma ,FPS ..etc but it is worth noting that the issue is not with all effects but only few of them ( e.g waves seems perfect) ., the one listed by the OP is somewhat still the same , FX192 Water is also still the same .
192_water_v2

@netmindz
Copy link
Collaborator

Please test the patterns and report any bugs with them against

https://github.com/netmindz/animartrix/tree/OO

WLED just wraps this library. When I raised the issue of the colours of Water Stefan said it related to fact it was designed for SmartMatrix which has gamma correction

@netmindz
Copy link
Collaborator

Quick (untested) attempt at adding gamma correction #85

@netmindz
Copy link
Collaborator

Found what's wrong with water - the red is going negative and so flipping to full brightness

@netmindz
Copy link
Collaborator

Next daily build has fixed version of rgb_sanity_check

@netmindz
Copy link
Collaborator

Should all now be resolved I think @FragMenthor if you try this build https://github.com/MoonModules/WLED/actions/runs/6599717911 (scroll to the bottom for the assets)

@FragMenthor
Copy link
Author

Should all now be resolved I think @FragMenthor if you try this build https://github.com/MoonModules/WLED/actions/runs/6599717911 (scroll to the bottom for the assets)

Thanks!!! I will try tomorrow as soon as I can! 😀

@dosipod
Copy link

dosipod commented Oct 22, 2023

Yeah i have tested all the effects and no more superimposed color so issue is fixed

@FragMenthor
Copy link
Author

Should all now be resolved I think @FragMenthor if you try this build https://github.com/MoonModules/WLED/actions/runs/6599717911 (scroll to the bottom for the assets)

Thanks!!! I will try tomorrow as soon as I can! 😀

Should all now be resolved I think @FragMenthor if you try this build https://github.com/MoonModules/WLED/actions/runs/6599717911 (scroll to the bottom for the assets)

Thank you! All effects seem OK now! Great job! 👍 💯

@softhack007 softhack007 added the fxed in mdev fixed in latest mdev source code label Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fxed in mdev fixed in latest mdev source code
Projects
None yet
Development

No branches or pull requests

4 participants