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

Is there a way to keep and process chunks in WASM only #148

Closed
st1992 opened this issue Mar 5, 2021 · 6 comments
Closed

Is there a way to keep and process chunks in WASM only #148

st1992 opened this issue Mar 5, 2021 · 6 comments
Labels

Comments

@st1992
Copy link

st1992 commented Mar 5, 2021

If we pass a WASM function as the callback to the encoder, will it be good for my performance or will it not make any difference.

@chcunningham
Copy link
Collaborator

I expect it wouldn't make any difference. @ecmziegler to confirm

@padenot
Copy link
Collaborator

padenot commented May 5, 2021

How would that work? WebAssembly/design#1162 isn't solved, so there is a copy, so it's slower. Of course then you can use WASM to process the encoded packet, but this really is orthogonal to Web Codecs.

@aboba
Copy link
Collaborator

aboba commented May 18, 2021

@sumeet-tiwari-samespace Applications which formerly used WASM for video encode/decode are moving to WebCodecs, because of the improved performance. So while WASM is viable for audio encode/decode, it will not improve performance in video encode/decde, even with WASM SIMD and a Dedicated Worker.

@padenot
Copy link
Collaborator

padenot commented May 19, 2021

I think this is about post-processing the packets/frames in WASM, right ? Not really encoding/decoding in WASM.

@st1992
Copy link
Author

st1992 commented Jul 28, 2021

yes @padenot

@padenot
Copy link
Collaborator

padenot commented Jul 28, 2021

Then I'd just say that it works today, but it might work better at some point if/when WebAssembly/design#1162 is designed and implemented and shipped.

Because this is not a Web Codecs issues, I'm closing this, but please feel free to open more issues if needed. Thanks!

@padenot padenot closed this as completed Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants