-
Notifications
You must be signed in to change notification settings - Fork 114
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
degradationPreference is under-specified #2248
Comments
As for Chrome, it's not exposed right now since we are still tuning the balanced degradation preference which is supposed to be the default. We also default internally to "maintain framerate", so we'd be changing the current default behavior in lots of apps, which is tricky to handle. I don't think we've had any issues interpreting the behavior of the flag though. "Maintain framerate" will try to adapt the frame size to keep the transmission smooth, "maintain resolution" will not change the frame size but drop frames as needed. |
@Orphis Is it accurate to say that in the Chrome implementation |
@aboba I believe that is correct. |
Some of the concerns described here may also apply to |
When degradationPreference is moved to webrtc-extensions, this issue can move there as well. |
degradationPreference
appears to have been implemented differently between browsers, but thedifferences are not being picked up in the WPT test, which only tests getting and setting values.
Currently Section 5.2.2 says:
"When bandwidth is constrained and the
RTCRtpSender
needs to choose between degrading resolution or degrading framerate,degradationPreference
indicates which is preferred."In the current Edge implementation,
degradationPreference
is interpreted as providing a content-hint to the encoder, allowing it to differentiate between applications desiring a bias toward "detail" (adegradationPreference
of "maintain-resolution" or "motion" (adegradation-preference
of "maintain-framerate").However my understanding is that other browsers don't implement
degradationPreference
this way.The text was updated successfully, but these errors were encountered: