search

muaz-khan / RTCMultiConnection

RTCMultiConnection is a WebRTC JavaScript library for peer-to-peer applications (screen sharing, audio/video conferencing, file sharing, media streaming etc.)
https://muazkhan.com:9001/
MIT License
2.53k stars 1.37k forks source link

A WebRTC newbie struggles.... #431

Open rayj00 opened 6 years ago

rayj00 commented 6 years ago

I'm still scratching my head trying to figure out what Muaz's goal is in all of this? Granted, I admire his ambition and him providing all of this neat code, but since there are no comments in the code, and I consider myself very green with WebRTC and Javascript, I am struggling to get anything to work correctly. Right now I am playing with Multiple-Broadcasters-Many Viewers. Very complex code. It does not work as I might expect and there are major issues. As Muaz rarely answers questions, is it entirely up to us to find and fix bugs? If that's the case, I guess I'm screwed as it will take me quite a long time to figure out the code. Reporting the issues are not getting anywhere. Aren't demos supposed to work by default?

Wemperer commented 6 years ago

Though the codebase for WebRTCMulticonnect is fairly extensive, it is not incredibly complex. The code is commented and every demo I've used works as long as you set everything up as instructed (which is minimal).

Thank you Mauz for your hard work. The work you've done has changed my life in ways you'll never know.

rayj00 commented 6 years ago

I don't know what code you are looking at but RTCMulticonnection is not commented. Neither are the couple of demos I am playing with. Don't get me wrong, I admire Muaz for the intensive work he is doing. And I realize he is doing this in his spare time. I'm sure he is in demand in the WebRTC world.

It's just very difficult for me as I do not have the Javascript or WebRTC experience as do most of you. So I am running various tests on the demo I am playing with and reporting to Muaz. The Multi-Broadcast-Many-Viewers has a bug. Scalable-Broadcast has multiple bugs.