TobKra96 / music_led_strip_control

Audio visualization for LED strips in real-time with web interface on a raspberry pi.
https://tobkra96.github.io/music_led_strip_control/
MIT License
297 stars 64 forks source link

Flickering #128

Closed llamatude closed 3 years ago

llamatude commented 3 years ago

I am using https://github.com/jgarff/rpi_ws281x for another project and it works well no issues. For some reason with MLSC i am getting flickering when on solid colors. I assume its doing it on the music as well but so much is going on i can tell. Everything else seems to work well just this odd flicker.

Teraskull commented 3 years ago

Hi,

Can you write the steps to reproduce the issue?

How many devices are created and does the flickering continue if only one device is active?

Also, are you using the master or dev_2.2 branch?

llamatude commented 3 years ago

If i turn it on solid color or anything really it just flickers.

I am running raspberry pi no remote modules yet

i am running master.. From what i can tell is when it updates the colors is when i get the flickering.

Teraskull commented 3 years ago

Could you please try running https://github.com/TobKra96/music_led_strip_control/tree/dev_2.2 and tell if the issue persists?

No other setup is required except for cloning/pulling the repo, switching the branch to dev_2.2, and installing pip modules with:

sudo pip3 install -r requirements
llamatude commented 3 years ago

still having the issue.. I made a small video for you to see:

https://youtu.be/BcDKlZWVmls

the code i am using that works no flicker is this: def Blank(strip): SetAll(strip, Color(255, 0, 0)) strip.show() I feel like its a power issue I have 2 strips at 54 lights per strip. I set the lights to 54 but I think its trying to send the signal to 124 lights. I do have a SPI Signal Amplifier Repeater. https://www.amazon.com/gp/product/B07LGM1694

llamatude commented 3 years ago

NVM this is a me issue... i just did a test with and i am getting the same flicker.. odd now time to trace this down! sorry for the false alarm.

for i in range(0, 3):
    #Fade In.
    for j in range (0, 256):
        if i == 0:
            SetAll(strip, Color(j, 0, 0))
        elif i == 1:
            SetAll(strip, Color(0, j, 0))
        elif i == 2:
            SetAll(strip, Color(0, 0, j))
        strip.show()
    #Fade Out.
    for j in range (256, 0, -1):
        if i == 0:
            SetAll(strip, Color(j, 0, 0))
        elif i == 1:
            SetAll(strip, Color(0, j, 0))
        elif i == 2:
            SetAll(strip, Color(0, 0, j))
        strip.show()
Teraskull commented 3 years ago

No problem, thanks for testing and confirming!

TobKra96 commented 3 years ago

I think the issue can be closed :)