Skip to content

Comments

[use case] Noise Suppression#61

Merged
huningxin merged 3 commits intowebmachinelearning:masterfrom
ibelem:nlp-use-cases
Jun 2, 2020
Merged

[use case] Noise Suppression#61
huningxin merged 3 commits intowebmachinelearning:masterfrom
ibelem:nlp-use-cases

Conversation

@ibelem
Copy link
Contributor

@ibelem ibelem commented May 18, 2020

@anssiko Thanks for improving the WebNN Spec, for use cases we may lack of some NLP or RNN use cases, for example the Noise Suppression becomes popular than before. PTAL.


Preview | Diff

@huningxin
Copy link
Contributor

Thanks @ibelem for contributing this use case.

According to resolution 02 of WebML CG Teleconference – 28 May 2020

RESOLUTION: Adopt the noise suppression use case, evaluate rnnoise model for required ops

this PR is going to be merged.

@ibelem , could you please solve the conflicts before that?

@ibelem
Copy link
Contributor Author

ibelem commented May 29, 2020

Thanks @huningxin, the conflict should be resolved, PTAL.

@ibelem
Copy link
Contributor Author

ibelem commented May 29, 2020

The conflict should be resolved, @huningxin PTAL.

@huningxin
Copy link
Contributor

Thanks @ibelem , I am going to merge it.

@huningxin huningxin merged commit d7b8e8d into webmachinelearning:master Jun 2, 2020
@ibelem
Copy link
Contributor Author

ibelem commented Jun 3, 2020

@huningxin Many thanks!

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