Skip to content
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

Transmit Stereo Audio #51

Open
iotku opened this issue Jun 5, 2020 · 2 comments
Open

Transmit Stereo Audio #51

iotku opened this issue Jun 5, 2020 · 2 comments

Comments

@iotku
Copy link

iotku commented Jun 5, 2020

Since mumble-voip/mumble#4209 has been merged (and will most likely be included in the Mumble 1.4.0 release) it would be nice to have the ability to transmit stereo audio implemented into gumble.

Presumably, at a quick uninformed glance, all that would need to change would be setting AudioChannels to 2 although it might be more complicated than that.

AudioChannels = 1

Supposedly the "opus decoder can also automatically convert a stereo stream into a mono stream, so even if we transmit a stereo stream to an older version of mumble, it would still work." according to the author of the PR.

Regardless, it is probably sensible to default to mono still (since 1.4.0 hasn't released yet) with a configurable option to transmit in stereo.

@ghost ghost added the feature-request label Jun 5, 2020
@iotku
Copy link
Author

iotku commented Feb 28, 2022

Through random poking around I was somewhat successful in transmitting stereo audio without major glitches by changing in audio.go

AudioDefaultFrameSize = (AudioSampleRate / 100) * AudioChannels
AudioChannels = 2

and in writeAudio() modifying raw to
raw, err := encoder.Encode(a, len(a)/AudioChannels, dataBytes)

But there's some bleed through from the left & right channels to each other and I'm still pretty out of my depth here.

@iotku
Copy link
Author

iotku commented Feb 28, 2022

It appears it's due to mumble client settings, Enabling positional audio and headphones (under positional audio) check-boxes in the audio output settings appears to solve the issue with the bleeding

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant