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

Significantly optimized the sound latency performance #3

Merged
merged 1 commit into from
Sep 21, 2024

Conversation

xiaoxuhuanxiang
Copy link
Contributor

Significantly optimized the sound latency performance, essentially refining the ring buffer algorithm to the extreme, ensuring good performance even on older devices (such as the A9X processor). For newer devices, turning off and then turning the sound back on once may achieve even lower latency.

@hugeBlack
Copy link
Owner

Thank for your contribution! I'll look into this later. By the way, have you find out why bit rate is super low when using the Parsec SDK? It seems that this issue can only be ameliorated by using an USB ethernet adapter.

@xiaoxuhuanxiang
Copy link
Contributor Author

Thank for your contribution! I'll look into this later. By the way, have you find out why bit rate is super low when using the Parsec SDK? It seems that this issue can only be ameliorated by using an USB ethernet adapter.

I have not encountered this issue here; I chose the Parsec SDK. Tested within a local area network, at 4K resolution with H.265 encoding. The bitrate is perfectly normal, and the clarity and smoothness are both excellent (the device is an iPhone 12 Pro Max). In contrast, another SDK is very laggy and unusable.

@hugeBlack hugeBlack merged commit 05f4386 into hugeBlack:main Sep 21, 2024
3 checks passed
@hugeBlack
Copy link
Owner

Hi, since you haven't enabled issue in your repo, I just ask here. Were you able to render with Metal? It seems that there's a bug with Parsec SDK that it will pass nil to setRenderPipelineState that will cause Set Render Pipeline State Validation renderPipelineState must not be nil. error and crash the app. Did you find a solution?

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.

2 participants