Skip to content

34g4m disable1#3732

Merged
beastoin merged 2 commits into
mainfrom
34g4m_disable1
Dec 12, 2025
Merged

34g4m disable1#3732
beastoin merged 2 commits into
mainfrom
34g4m_disable1

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

No description provided.

@beastoin beastoin marked this pull request as ready for review December 12, 2025 08:38
@beastoin beastoin merged commit 7610399 into main Dec 12, 2025
1 check passed
@beastoin beastoin deleted the 34g4m_disable1 branch December 12, 2025 08:38
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request modifies the speech-to-text streaming utility. It disables single-language support for Deepgram's Nova 3 model by hardcoding an empty list of languages, which is a critical issue as it can break functionality in different deployment environments. This should be controlled by a configuration flag. Additionally, the PR introduces a bug where it incorrectly disables the keywords feature for all non-English languages while attempting to restrict the keyterm feature to English only. My review provides suggestions to address both of these issues.

Comment thread backend/utils/stt/streaming.py
Comment thread backend/utils/stt/streaming.py
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
* Limits keywords for english only

* Disable nova-3 single langs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant