-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Feature : Soniox STT Provider Implemented #116
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
base: main
Are you sure you want to change the base?
Conversation
Surya-sourav
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Soniox Provider Integration
- Implemented SonioxSTTSession in soniox.js with real-time transcription using the Soniox API.
- Added a close() method to match the expected STT session interface.
- Registered Soniox in the PROVIDERS object in factory.js with its model.
- API Key Validation
- Updated modelStateService.js to handle Soniox in validateApiKey, including format validation and proper state updates.
- Ensured Soniox API key is saved and the provider/model is auto-selected after validation.
IPC & State Management
- Removed duplicate model:validate-key IPC handler from windowManager.js to avoid Electron handler conflicts.
- Ensured only modelStateService.js manages API key validation and state.
UI/UX
- Soniox now appears as a selectable STT provider in the configuration UI.
- Fixed "Unknown provider" error for Soniox in the UI.
|
Hi @Surya-sourav |
|
@samtiz Let me check it once again and reconfirm. Thanks for testing ! |
|
@samtiz Actually i got the issue , Soniox expects the audio in a WAV format but Glass app is likely sending Raw PCM , so let me just fix this asap and re pull the PR |
|
@samtiz You can test it now , I hope it wont be an issue after the fix. Thanks :) |
|
@samtiz @entry-sanio @jhyang0 @ekim425 Merge Conflicts are resolved , its ready to get Merged ! |
|
@samtiz @entry-sanio @ekim425 Please confirm about this PR ! |
|
@samtiz @entry-sanio Any updates on this PR from your side? |
|
@samtiz @entry-sanio Guys , this PR is getting old and currently is 14 commits behind the latest Upstream. What are your suggestions ? Ik u must be busy with other important features & bug fixes but your decision would help here. |
|
@samtiz @entry-sanio : I know you are busy! I really implore you though to have a look at this PR - it would be quite a big jump in quality for Glass : ) |
Integrate Soniox as STT Provider in Glass App
This PR adds support for Soniox as a speech-to-text (STT) provider in the Electron-based Glass app. Users can now select Soniox as their STT provider, validate their API key, and use Soniox for real-time transcription, similar to Whisper.
Summary of Changes
Soniox Provider Integration
API Key Validation
IPC & State Management
Related Issue
Contributor's Self-Review Checklist
Please check the boxes that apply. This is a reminder of what we look for in a good pull request.
Additional Context (Optional)
** Logs :**
[ModelStateService] Selected STT model from newly configured provider soniox: en_v2
[ModelStateService] State saved for user: default_user
[ModelStateService] Current Selection -> LLM: claude-3-5-sonnet-20241022 (Provider: anthropic), STT: en_v2 (Provider: soniox)
[ModelStateService] Current Selection -> LLM: claude-3-5-sonnet-20241022 (Provider: anthropic), STT: en_v2 (Provider: soniox)
[ModelStateService] API key for soniox is valid.
@samtiz @entry-sanio @jhyang0 @ekim425 Please Review the PR & give me feedback.
I would also review the PR once again from my end.