search

esphome / home-assistant-voice-pe

Home Assistant Voice PE
Other
72 stars 10 forks source link

[micro_wake_word] Bugfix: New spectrogram features should go to the back of the queue #79

Closed kahrendt closed 2 months ago

kahrendt commented 2 months ago

In #61, I modified how the spectrogram features were transferred from the preprocessor task to the inference task, where it uses a queue instead of a ring buffer. I mistakenly sent the newest features to the front of the queue. If the inference task ever fell even slightly behind, then this would result in the spectrogram features being processed out of order. This explains the sudden decrease in wake word performance! This PR sends the newest features to the back of the queue so everything is processed in the correct order.

github-actions[bot] commented 2 months ago

Firmware built successfully! :tada:

Download and extract the firmware to install with https://web.esphome.io

Make sure to choose esphome-voice-kit-esp32s3/esphome-voice-kit-esp32s3.factory.bin.