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

Stop putting the session info (all channels of all participants) into… #39

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

christofmuc
Copy link
Owner

… each audio block, because that increases the block size of the audio stream a lot when more participants join the session. It might even increase the block size beyond the maximum frame size, which makes the audio go kaputt. Instead, send a separate package with a reduced frame rate to the other clients. This makes the peak meteres be less real time, but it saves a lot in terms of audio package size and allows us to turn on FEC again!

… each audio block, because that increases the block size of the audio stream a lot when more participants join the session. It might even increase the block size beyond the maximum frame size, which makes the audio go kaputt. Instead, send a separate package with a reduced frame rate to the other clients. This makes the peak meteres be less real time, but it saves a lot in terms of audio package size and allows us to turn on FEC again!
…eter changes to the server. Not enjoying flatbuffers enough to put all the parameters into individual flatbuffer messages, we'll be just using a string with JSON here. This is low frequency data, therefore that overhead won't matter.
…e FEC setting, that is now coupled with the Heal checkbox on the client side.
…libraries needed to build only the server on Linux
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.

1 participant