Skip to content

Commit

Permalink
CHANGE(client): Disable RNNoise by default
Browse files Browse the repository at this point in the history
In the 1.4.230 release, we enabled RNNoise by default as there have been
lots of reports of how well it works. However, after the release we saw
numerous reports complaining about bad audio quality, which was traced
back to having RNNoise enabled.

For some reason the outcome of having RNNoise enabled is very different
in different scenarios. Sometimes it works miraculously well and other
times it worsens the audio quality to an unbearable level.

There seems to be a higher chance of RNNoise messing up, when using
Windows, but the same effect has also been observed on Linux. It might
also depend on the quality of the used microphone (better quality = less
noise -> RNNoise starts doing weird stuff), but we don't have any
definitive clues yet.

Because of this, we will change the default noise cancelling mode back
to Speex so that everyone for whom RNNoise actually works, cna enable
it, but we don't kill the experience for many folks by using an
unsuitable default value.

Fixes #5448
  • Loading branch information
Krzmbrzl committed May 11, 2022
1 parent e1c7ac4 commit 7885104
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/mumble/Settings.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -499,9 +499,6 @@ Settings::Settings() {
qRegisterMetaType< Search::SearchDialog::ChannelAction >("SearchDialog::ChannelAction");


#ifdef USE_RNNOISE
noiseCancelMode = NoiseCancelRNN;
#endif
#ifdef Q_OS_MACOS
// The echo cancellation feature on macOS is experimental and known to be able to cause problems
// (e.g. muting the user instead of only cancelling echo - https://github.com/mumble-voip/mumble/issues/4912)
Expand Down

0 comments on commit 7885104

Please sign in to comment.