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
Disable panner node for Android #5291
Conversation
We might try switching to |
Oops, did not mean to close the PR |
Hm, from the test in #5057,
Good point. If we force to disable it, we may be able to start to force only on Android phones. |
I agree it's unfortunate we have to completely turn off positional audio for all android devices. I don't expect this to be fixed on Android or Chrome any time soon. Perhaps they will take months to fix it. If the fix ends up being in Android, then it's possible that many existing users with old phones may not receive the update for even longer, if at all. If possible, perhaps we could disable positional audio on Android if there are more than 15 users in the room. That way users who often use Hubs with a small number of people will not be affected. |
At first I thought of that option but I started to feel I want to avoid because
So I'm thinking of disabling on Android Phone devices regardless of the number of people. I may change my mind if the workaround code which disable positional audio depending on the number of used panner nodes, can be simple. |
Currently Android seems to have a panner node audio problem. We already reported it. Until it will be resolved, we force to disable it as short-term workaround. We disable only for Android phones because we haven't heard the problem on VR devices so far.
3e9f592
to
ac022f7
Compare
I have updated the PR to disable it only for Android Phones. Some thoughts.
|
Closing in favor of #5540 |
Currently Android seems to have a panner node audio problem. With 10 or many Panner nodes (the number depends on Android OS version or device. The problem seems to happen more easily on older or lower-end devices) the audio can be crackled or broken. See #5057
We already reported it. Until it will be resolved on Android end, I would like to suggest to force to disable it as short-term workaround. The change is small. It may not be a perfect solution but it is a short-term workaround. I hope we can accept.