-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enabled some audioreactive effects for single pixel strips/segments #3942
Conversation
I think it is ok, but let's wait what @softhack007 says. |
@softhack007 could you look into this, please. IMO the changes are benign and I'd like to include them in b3 release scheduled for next week. |
I'm on it 馃槂 overall looks good, thanks @gaaat98 for doing the "zero D" checks. I'll try them on my own setup in next days, plus i'll review the code to be sure there is no danger of running into invalid pointers or div/0 situations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good :-)
Please just delete the commented-out checks like if (SEGLEN == 1) return mode_static();
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃憤
Enabled some audioreactive effects for single pixel strips/segments
"Solution" to #3937
Enabled Juggles, Noisefire, Plasmoid, DJ Lights, Freqmatrix, Freqwave, Noisemove, Rocktaves, and Waterfall effects (I chose the ones that looked the best to me) to be selected for single pixel segments/LED strips.
Some effects (noisefire, plasmoid) needed just the metadata to be updated, the others also needed the
if (SEGLEN == 1) return mode_static();
check to be disabled, I verified that removing the check does not create problems in the functions (hopefully 馃槄, see comments in code).