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

Fixing VAD #103

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

Fixing VAD #103

wants to merge 2 commits into from

Conversation

PrinceBaghel258025
Copy link
Member

@PrinceBaghel258025 PrinceBaghel258025 commented Nov 6, 2024

  • disable record button until model file loads
  • efficient hash table for handling text chunks
  • Implement user preferred language for generating answers

Copy link

sweep-ai bot commented Nov 6, 2024

Hey @PrinceBaghel258025, here are examples of how you can ask me to improve this pull request:

@Sweep Fix the CI errors.
@Sweep Add unit tests for the `VadAudio` component to verify:
1. The record button is disabled when `vad.loading` is true
2. The record button is disabled when `isTranscribing` is true and `vad` is not listening
3. The `onStartListening` and `onStopListening` callbacks are called correctly

📖 For more information on how to use Sweep, please read our documentation.

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