-
Notifications
You must be signed in to change notification settings - Fork 181
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
Web Workers #34
Comments
Yeah you can't reference DOM elements in worker scripts, so unfortunately On Mon, Dec 7, 2015 at 2:59 PM, Josh Beckwith notifications@github.com
|
Ok, so we definitely need to call |
I've just tried it. This method of delayed encoding works perfectly. It's almost an order of magnitude faster, during recording (at least for my application). Current Whammy: https://youtu.be/QK9AOAvGMsw Delayed encoding: https://youtu.be/99Kob5Xv0Yk |
delayed encoding turns out to be a huge benefit while recording video on mobile. Built in MediaRecorder api results in choppy video, Whammy saved the day here. However, the encoding process really stalls the interface afterwards. I'd like to point to the OffscreenCanvas interface:
This is not yet there, but may provide even further improvements during the compile phase by actually using webworkers for that |
Recording at high res / fps is pretty choppy. Is there some way we can improve this by offloading the
.toDataURL()
call to web workers? I assume that is where the bottleneck is, but I don't think you can reference dom elements in worker scripts.Thoughts?
The text was updated successfully, but these errors were encountered: