Jan/WLED_MM_Infinity
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

esp32 audioreactive bugfix

Browse Source 
small mistake, huge impact: the right way is to first create our FFT task, then check if task creation was successful.
This commit is contained in:
Frank
2023-08-24 22:21:48 +02:00
committed by GitHub
parent 6379a61828
commit 63375f5c84
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
usermods/audioreactive/audio_reactive.h
View File

@@ -1724,11 +1724,11 @@ class AudioReactive : public Usermod {
delay(250); // give microphone enough time to initialise
if (!audioSource) enabled = false; // audio failed to initialise
if (FFT_Task == nullptr) enabled = false; // FFT task creation failed
#endif
if (enabled) onUpdateBegin(false); // create FFT task
if (enabled) disableSoundProcessing = false; // all good - enable audio processing
if (enabled) onUpdateBegin(false); // create FFT task, and initailize network
#ifdef ARDUINO_ARCH_ESP32
if (FFT_Task == nullptr) enabled = false; // FFT task creation failed
if((!audioSource) || (!audioSource->isInitialized())) { // audio source failed to initialize. Still stay "enabled", as there might be input arriving via UDP Sound Sync
#ifdef WLED_DEBUG
DEBUG_PRINTLN(F("AR: Failed to initialize sound input driver. Please check input PIN settings."));
@@ -1740,6 +1740,7 @@ class AudioReactive : public Usermod {
USER_PRINTLN(F("AR: sound input driver initialized successfully."));
}
#endif
if (enabled) disableSoundProcessing = false; // all good - enable audio processing
// try to start UDP
last_UDPTime = 0;
receivedFormat = 0;