-
Notifications
You must be signed in to change notification settings - Fork 237
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
RnNoise v0.2 #194
RnNoise v0.2 #194
Conversation
Ensure unique LV2 URI string when suffix is provided, to ensure both mono and stereo variants function.
Unique LV2URI
There are additional changes beside rnnoise update, could you add why change was made to the respective commits? |
They are all from the rnnoise update except for the google style and lv2 bug. |
@pengzhendong |
Sorry, I don't know. You could refer to the original repository: https://github.com/xiph/rnnoise |
@XCroatoanX For personal testing purposes I created one file with part of song and processed it with same settings, but one from old RNNoise 1.03 and second from RNNoise + this PR The difference is obvious in first 5-10 seconds |
Wow, thank you very much, the difference is indeed day and night, can’t wait to have the update on Arch! |
If you can't wait, like me, maybe you can build it for yourself like I did, by forking https://github.com/pengzhendong/noise-suppression-for-voice and using Github Actions workflow |
I built the LADSPA plugin with MSVC using a GitHub Actions workflow, but it failed. https://github.com/pengzhendong/noise-suppression-for-voice/actions/runs/9121513325/job/25080780130 |
@pengzhendong But we have logs from GH actions from this repo after you created this PR Seems like, yesterday it uses different MSVC version |
It doesn't build all plugins after I created this PR:
|
@pengzhendong |
I see OBS also have issues with rnnoise compilation with newer compilers and they fixed it |
Not the same issue. The issue is about the LADSPA plugin, not rnnoise. |
Thanks @pengzhendong, I updated rnnoise myself since I wanted to make the update cleaner: https://github.com/werman/noise-suppression-for-voice/commits/develop/ Downgraded MSVC to 2019 to solve compiler crash. For now Could someone test the build (https://github.com/werman/noise-suppression-for-voice/actions/runs/9140756650)? |
I can try and test on arch, if needed |
I don't normally use VST, so I'm not a good test for that (because I would not really know if it failed) but I did test the ladspa version on tumbleweed, and it works great! Thanks SOOOO much for updating this repo! It's a really big deal for people using system packages, who nearly always get their sources from here rather than a fork. Also it is nice to know that you are alive and well :D Thanks so much for all the annoying noises we didn't have to listen to because of you! <3 |
FYI I built this as an opensuse package, with |
@pallaswept Thanks! |
No description provided.