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

There is no muxId in RTCRtpEncodingParameters #528

Closed
ibc opened this issue May 2, 2016 · 4 comments
Closed

There is no muxId in RTCRtpEncodingParameters #528

ibc opened this issue May 2, 2016 · 4 comments

Comments

@ibc
Copy link
Contributor

ibc commented May 2, 2016

In http://ortc.org/wp-content/uploads/2016/03/ortc.html#idl-def-RTCRtpParameters:

muxId of type DOMString, defaulting to ""

[...] Note that including muxId in RTCRtpParameters rather than in RTCRtpEncodingParameters implies that if it is desired to send simulcast streams with different muxId values for each stream, then multiple RTCRtpSender objects are needed.

But I cannot find muxId within RTCRtpEncodingParameters... May be it means the encodingId field, but that is related to the RID header extension which, AFAIK, it's not the same as the MID header extension.

@aboba
Copy link
Contributor

aboba commented May 2, 2016

In ORTC, the equivalent of RTCRtpEncodingParameters.rid in WebRTC 1.0 is
RTCRtpEncodingParameters.encodingId.

In ORTC the equivalent of RTCRtpTransceiver.mid in WebRTC 1.0 is RTCRtpParameters.muxId.

In both ORTC and WebRTC 1.0, there only a single muxId/mid per receiver or sender.

@ibc
Copy link
Contributor Author

ibc commented May 2, 2016

Thanks, but I still consider that the spec is wrong as it clearly says "Note that including muxId in RTCRtpParameters rather than in RTCRtpEncodingParameters", but it is clear that there is no muxId in RTCRtpEncodingParameters.

I would replace the above quoted text with:

When it comes to simulcast, including muxId in RTCRtpParameters rather than different encodingId in RTCRtpEncodingParameters implies that multiple RTCRtpSender objects with different muxId are needed.

@aboba
Copy link
Contributor

aboba commented May 4, 2016

In addition to a change to the referred to text, a note in the WebRTC compatibility section appears needed.

@ibc
Copy link
Contributor Author

ibc commented May 4, 2016

if it is desired to send simulcast streams with different muxId values for each stream, then multiple RTCRtpSender objects are needed.

I assume that a ORTC browser cannot receive simulcast in this fashion (it would require multiple RtpReceivers so multiple generated MediaStreamTracks). Am I right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants