Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Whisper to hear players correctly #27

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix Whisper to hear players correctly #27

wants to merge 2 commits into from

Conversation

S0L0GUY
Copy link
Owner

@S0L0GUY S0L0GUY commented Jan 26, 2025

Fixes #26

Fix the get_speech_input method in classes/whisper.py to correctly detect silence and stop recording.

  • Silence Detection

    • Change silence_duration from milliseconds to seconds.
    • Adjust the condition to stop recording after detecting sufficient silence.
  • Error Handling in nova.py

    • Add try-except block to handle errors during the transcription process.
    • Print error messages and set typing indicator on error.

S0L0GUY and others added 2 commits January 26, 2025 14:05
Fixes #26

Fix the `get_speech_input` method in `classes/whisper.py` to correctly detect silence and stop recording.

* **Silence Detection**
  - Change `silence_duration` from milliseconds to seconds.
  - Adjust the condition to stop recording after detecting sufficient silence.

* **Error Handling in `nova.py`**
  - Add try-except block to handle errors during the transcription process.
  - Print error messages and set typing indicator on error.
@S0L0GUY S0L0GUY added the 🧪 Needs to be tested This has not been tested yet label Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧪 Needs to be tested This has not been tested yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Whisper Broken
1 participant