Skip to content

Conversation

@gui217
Copy link

@gui217 gui217 commented Dec 6, 2025

Add RNNoiseFilter implementation for real-time noise suppression using
RNNoise neural network via pyrnnoise library. The filter buffers audio
to match RNNoise's required frame length (480 samples at 48kHz) and
processes it in chunks.

  • Add RNNoiseFilter class with start/stop/process_frame/filter methods
  • Add comprehensive test suite covering noise reduction, passthrough
    when disabled, buffering behavior and. resampling
  • Add rnnoise optional dependency (pyrnnoise~=0.2.0) to pyproject.toml

In order to use simply:

TransportParams(
             ...
            audio_in_filter=RNNoiseFilter(),
            ...
        ),

@gui217 gui217 marked this pull request as ready for review December 6, 2025 19:09
@gui217 gui217 force-pushed the feat/rnnoise branch 2 times, most recently from 9eee540 to edd1aa5 Compare December 7, 2025 14:48
@aconchillo aconchillo self-requested a review December 8, 2025 05:45
@aconchillo
Copy link
Contributor

Awesome! Thank you @gui217 ! Do you mind rebasing and updating uv.lock?

@gui217
Copy link
Author

gui217 commented Dec 8, 2025

@aconchillo rebased and fixed uv.lock 🙏

@aconchillo
Copy link
Contributor

@aconchillo rebased and fixed uv.lock 🙏

Thanks! Something is off with the UT.

@gui217
Copy link
Author

gui217 commented Dec 9, 2025

@aconchillo rebased and fixed uv.lock 🙏

Thanks! Something is off with the UT.

should be fixed now - how can i re-trigger the workflows?

@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 11 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pipecat/audio/filters/rnnoise_filter.py 83.33% 11 Missing ⚠️
Files with missing lines Coverage Δ
src/pipecat/audio/filters/rnnoise_filter.py 83.33% <83.33%> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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