Skip to content

Add manual mode to noise reduction + add noise reduction threshold #1007

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

Merged

Conversation

digisomni
Copy link
Member

@digisomni digisomni commented Feb 9, 2021

Generally, it did a damn great job of filtering out my keystrokes though. Users that find a good number for the threshold I think will have a good time with this. Pretty much all voice applications (ever) have manual noise gates. I always use manual mode, it helps for tuning what I want to convey as an experience instead of leaving it to algorithms.

image

@digisomni digisomni added the enhancement New feature or request label Feb 9, 2021
@digisomni digisomni marked this pull request as ready for review February 11, 2021 10:59
@digisomni
Copy link
Member Author

This is ready for review now.

@digisomni digisomni added this to the 2021.1.1 Eos Release milestone Feb 11, 2021
@SilverfishVR
Copy link
Contributor

I seems to work, but I could not find anyone online on master so I did not actually talk to anyone 😄

@digisomni
Copy link
Member Author

I seems to work, but I could not find anyone online on master so I did not actually talk to anyone 😄

I actually talk to myself, no joke. I'd just listen on my other client to see at what points I'd hear me. :)

@digisomni digisomni added the scripting api change Change is made in the scripting API label Feb 13, 2021
@digisomni digisomni linked an issue Feb 13, 2021 that may be closed by this pull request
@ctrlaltdavid
Copy link
Collaborator

UI suggestion: Change "Noise Reduction: Manual/Automatic" to "Manual Noise Reduction".
This would make it clearer that the toggle enables manual noise reduction?

@digisomni
Copy link
Member Author

Currently it's a switch that controls the value of "noiseValueAutomatic" , but I could reverse it and rename it probably, yeah.

"Noise Reduction: Manual/Automatic" -> "Manual Noise Reduction"
@ctrlaltdavid ctrlaltdavid added needs testing (QA) The PR is ready for testing and removed needs CR (code review) labels Feb 24, 2021
@digisomni digisomni added the CR Approved At least one code reviewer has approved the PR. label Feb 25, 2021
@Aitolda
Copy link
Collaborator

Aitolda commented Feb 26, 2021

Threshold seems to do what it's supposed to do.. HOWEVER... the "test your mic" audio feed seems to be pre-noise reduction so this is probably not the most ideal place in the signal flow. Ideally the feed played while you "test your mic| would be after noise reduction.

@Aitolda
Copy link
Collaborator

Aitolda commented Mar 3, 2021

Mic listen now working post-noise reduction. Seems to be working as intended.

@digisomni digisomni added QA Approved The PR has been tested successfully. and removed needs testing (QA) The PR is ready for testing labels Mar 3, 2021
Fix default value for "NoiseReductionAutomatic" to true.
@digisomni digisomni merged commit 672bcee into vircadia:master Mar 4, 2021
@digisomni digisomni deleted the feature/manual-noise-reduction-gate branch March 4, 2021 02:19
@Aitolda
Copy link
Collaborator

Aitolda commented Mar 4, 2021

Update tested. Manual noise reduction no longer default (starts automatic). Otherwise continues to function as before.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR Approved At least one code reviewer has approved the PR. enhancement New feature or request QA Approved The PR has been tested successfully. scripting api change Change is made in the scripting API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Microphone noise gate slider
5 participants