Skip to content

Add Ai-coustics support to noise-canceller#2

Open
codeSTACKr wants to merge 1 commit intomainfrom
jesse/add-ai-coustics
Open

Add Ai-coustics support to noise-canceller#2
codeSTACKr wants to merge 1 commit intomainfrom
jesse/add-ai-coustics

Conversation

@codeSTACKr
Copy link
Copy Markdown

  • Integrated Ai-coustics audio enhancement models (aic-quail-l and aic-quail-vfl) into the noise cancellation process.
  • Updated AudioFileProcessor to handle Ai-coustics credentials and stream information.
  • Modified command-line interface to include new filter options for Ai-coustics.
  • Updated README to reflect new features and usage instructions.
  • Bumped livekit-agents dependency and added livekit-plugins-ai-coustics to requirements.

- Integrated Ai-coustics audio enhancement models (`aic-quail-l` and `aic-quail-vfl`) into the noise cancellation process.
- Updated `AudioFileProcessor` to handle Ai-coustics credentials and stream information.
- Modified command-line interface to include new filter options for Ai-coustics.
- Updated README to reflect new features and usage instructions.
- Bumped `livekit-agents` dependency and added `livekit-plugins-ai-coustics` to requirements.
Comment thread noise-canceller.py
# Provide credentials to ai-coustics filter so it can authenticate.
# A separate token with a different identity is required so the Enhancer
# can connect to the room without conflicting with our existing connection.
if isinstance(self.noise_filter, AICousticsAudioEnhancer):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why is this needed? shouldn't it work automatically the same way krisp does?

Comment thread noise-canceller.py


# Provide stream info to ai-coustics filter now that we have the publication SID
if isinstance(self.noise_filter, AICousticsAudioEnhancer):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same question here - i don't understand why this is necessary for AIC but not for krisp

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.

2 participants