search

Nailik / rhasspy_mobile

Rhasspy is a voice assistant software. This repository implements the functionality of a Rhasspy satellite, with local wake word recognition.
65 stars 4 forks source link

Silence Detection #391

Closed arturpragacz closed 1 year ago

arturpragacz commented 1 year ago

I cannot get automatic silence detection to work correctly. Even though I have it set up for 2s silence detection time, it takes significantly longer than that to activate. Oftentimes it does not activate at all. Even when I set audio level threshold to some absurdly high value for testing, it does not activate in the time it should.

Nailik commented 1 year ago

which version are you using and what device, also android version would be interesting

arturpragacz commented 1 year ago

I have a Samsung phone with Android 13. I have now just installed the latest nightly version, so 0.4.9-90. What I see now is I can not get the silence detection to work at all. Every time the recording just goes on during complete silence, only stopping after about 10 seconds. I guess it is hitting the recording timeout set in the local dialog management.

Additional information, in case it's helpful: I do not have a wake word configured. I use the START_RECORDING broadcast message to activate Rhasspy*. I have also tested manual activation by pressing the button on the Home activity in the Rhasspy app, with the same result. I do not use MQTT, instead I use the HTTP remote endpoints. The webserver is disabled. So is the background service. I have sound and visual indication turned on.

* which seems to no longer work for some reason in the latest nightly I just installed

Nailik commented 1 year ago

jeah the broadcast is broken in the current nightly build. I'll check the problem and come back to you later.

Nailik commented 1 year ago

@arturpragacz about the broadcast i updated the readme, it change minimally see https://github.com/Nailik/rhasspy_mobile/pull/397 .

arturpragacz commented 1 year ago

Great! Have you had a chance to look into the silence detection issue as well?

Nailik commented 1 year ago

Jeah there is at least one bug where the minimum recording time is completely ignored. I rewrote the thing and currently implement unit tests for it so i can be sure it works as expected.

Nailik commented 1 year ago

@arturpragacz there is now a new nightly build available, i would be very happy if you could try it out and tell me if there are still issues for you