News for hackers. Stuff that matters.
Re:Show HN: WebRTC implementation for Python using Asyncio by Orphis on Monday October 08, 2018 @11:15AM (#18168188)
Nice work! As a WebRTC implementer (both native and in Chrome), I applaud the work!

But I can only notice quite a bit of mismatch between your APIs and the current spec. Do you plan on updating your implementation to match it?

Re:Show HN: WebRTC implementation for Python using Asyncio by jlaine on Monday October 08, 2018 @11:32AM (#18168340)
Is there any specific area you spotted which would need some love? Modern spec compliance is most definitely a goal (e.g. RTCRtpTransceiver has been there from the start, aiortc uses the "modern" SDP form for data-channels), so any pointers are greatly appreciated!

I've had some great interactions with the Mozilla crowd (hey Lennart, hey Nils) - looking forward to more of the same!

Re:Show HN: WebRTC implementation for Python using Asyncio by Orphis on Monday October 08, 2018 @11:48AM (#18168507)
Anything related to RTCRtpSendParameters (previously RTCRtpEncodingParameters) really.

For example, RTCRtpSender.getParameters is missing and the setParameters seems to be missing (unless that's the send function). And as far as I can tell, you are not supposed to call setParameters without getParameters first, so that's not spec compliant ;)

And of course, the sendEncodings property of the init parameters in addTransceiver is missing. It is required to setup simulcast (and later SVC when we standardize it).

Re:Show HN: WebRTC implementation for Python using Asyncio by jlaine on Monday October 08, 2018 @12:25PM (#18168790)
Duly noted, I've opened an issue to track this:

https://github.com/jlaine/aiortc/issues/76

aiortc's RTCPeerConnection indeed uses RTCRtpSender.send to apply parameters. This is directly inspired by ORTC:

http://draft.ortc.org/#dom-rtcrtpsender-send

Re:Show HN: WebRTC implementation for Python using Asyncio by Orphis on Monday October 08, 2018 @08:53PM (#18172552)
Also, my take on this is that you should probably not mix WebRTC and ORTC APIs. Those are quite different (even if there are similarities).

If you want to implement ORTC, fine, but you should probably do it in another package and make sure you don't expose fields in WebRTC that aren't part of the standard. Some fields (like the SSRC field) is not exposed for good reasons.