-
Notifications
You must be signed in to change notification settings - Fork 52
Evaluate noise suppression models for required ops #66
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
Comments
Hey @jmvalin your work is coming up yet again! @wchao1115, the WebNN API spec co-editor, was interested in exploring RNNoise a bit further in this context. So, to connect the dots, I thought it apropos to ping you here if any RNNoise questions arise, as, after all, you know more about RNNoise than anyone. |
@wchao1115 @kdavis-mozilla @anssiko Sure, let me know how I can help. |
Thanks @anssiko, @kdavis-mozilla, and @jmvalin. In addition to RNNoise I'd also like to propose NSNet, the baseline model for the Interspeech DNS Challenge. |
@jmvalin. From the implementation standpoint, both RNNoise and NSNet are hybrid DSP/ML based solution. Given that webnn is focusing on the deep learning part, how do you see the solution integrated into the web platform, especially on the signal processing part? I'm not up to date on the state of Web Audio implementation in the web browsers; do you think the level of support for FFT windowing in the browser is sufficient for RNNoise? |
I don't actually know the state of DSP in the browser. Being able to compute an FFT would certainly be nice, but it's not a showstopper. Similarly, having fast kernels for convolutions/correlations would be equally useful since RNNoise performs pitch analysis. Honestly what I've like to see at some point is a WebBLAS (plus FFT and convolution/correlation). That would probably cover most use cases -- including a big chunk of WebML. |
To date, we still need to define |
+1. And the @wchao1115 , if no duplication, I can work on a PR to add the |
Sounds good. Thanks @huningxin |
Noise suppression use case was adopted by WebNN API in #61 (thanks @ibelem!):
https://webmachinelearning.github.io/webnn/#usecase-noise-suppression
This issue is to discuss the next steps:
The text was updated successfully, but these errors were encountered: