-
Notifications
You must be signed in to change notification settings - Fork 14
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
Improve RTPBuffer #24
Comments
…closing #24 Signed-off-by:doe300 <stadeldani@web.de>
Improve the buffer:
|
Problems to solve:
|
We need to write test-cases for the three examples from the post above. |
The std::rand generator works terrible. It even should not have the name random number generator. Anyway, I have tested both buffers (RTPBuffer and RTPBufferAlternative) with an another random generator with 50% percent package lost. The RTPBufferAlternative buffer works quite good compared to RTPBuffer. |
AlternativeBuffer does not work on real enviroment (test with J0n8s). |
@Kaayy-J , How did the original RTPBuffer-implementation do in your test? |
It did quite well. We had a very good sound quality. |
rewrite one of the two buffer-implementations to solve all the problems in this ticket, including:
|
add minimum number of buffered package before playing audio
The text was updated successfully, but these errors were encountered: