search

muaz-khan / RecordRTC

RecordRTC is WebRTC JavaScript library for audio/video as well as screen activity recording. It supports Chrome, Firefox, Opera, Android, and Microsoft Edge. Platforms: Linux, Mac and Windows.
https://www.webrtc-experiment.com/RecordRTC/
MIT License
6.52k stars 1.75k forks source link

Please update MultiStreamsMixer to newer version v1.2.3 #724

Open vonweb opened 3 years ago

vonweb commented 3 years ago

Thanks for your great work, it helps a lot. MultiStreamsMixer in RecordRTC is v1.2.2, can you upgrade it to v1.2.3, in older version appendStreams creates new newStream, which lost some stream info left, top, width and height

trackedsupport commented 3 years ago

Does the MultiStreamsMixer not eat a ton of memory? I tried using it but had to pull it since it was causing my browser to crash. Even the demos cause my computer to run out of memory in a matter of 30 seconds.

vonweb commented 3 years ago

Does the MultiStreamsMixer not eat a ton of memory? I tried using it but had to pull it since it was causing my browser to crash. Even the demos cause my computer to run out of memory in a matter of 30 seconds.

It get stuck in both version, in the smaller version 1.2.2, the position info is lost, maybe it's because call drawVideosToCanvas using setTimeout, in my local env I use requestAnimationFrame, it works better