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

WebCodecs #433

Closed
3 tasks done
pthatcherg opened this issue Oct 18, 2019 · 6 comments
Closed
3 tasks done

WebCodecs #433

pthatcherg opened this issue Oct 18, 2019 · 6 comments
Assignees
Labels
Progress: review complete Resolution: satisfied The TAG is satisfied with this design Review type: CG early review An early review of general direction from a Community Group Topic: media Venue: Media WG Venue: WICG

Comments

@pthatcherg
Copy link

pthatcherg commented Oct 18, 2019

Hello TAG!

I'm requesting a TAG review of:

Further details:

You should also know that...

  • The review is early on in the design stage because we expect to learn a lot from early implementation feedback, so we'd like to get a rough design reviewed so that we can begin implementations and get that feedback to further refine the design.
  • A portion of the explainer and WebIDL could be done a separate standard from WebCodecs, namely the conversions from WHATWG streams to MediaStreamTracks (called TrackWriters and TrackReaders at the moment). Many use cases require these, but use cases outside of WebCodecs could also find these useful. For simplicity, we are considering such work inside the scope of WebCodecs, although it could be pulled out later.

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our GitHub repo for each point of feedback
@cynthia cynthia self-assigned this Nov 5, 2019
@torgo torgo added this to the 2019-11-19-telecon milestone Nov 5, 2019
@alice alice added the Review type: CG early review An early review of general direction from a Community Group label Dec 16, 2019
@dbaron
Copy link
Member

dbaron commented Jan 11, 2020

I've read through the explainer (twice, I think) and I don't have anything to complain about so far.

@alice alice removed this from the 2019-12-16-week milestone Jan 27, 2020
@cynthia
Copy link
Member

cynthia commented Mar 5, 2020

I've also looked at this and think it would be a useful addition to the platform. One question I do have is if the first deliverable could also support images - there is [1] HTMLImageElement.decode() but that really doesn't do exactly what one would expect. Could any of the editors comment on if this is possible?

CC: @steveanton, @padenot, @aboba, @pthatcherg

[1] https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/decode

@padenot
Copy link

padenot commented Mar 5, 2020

I'm pretty sure this would be possible. More and more, image codecs use video codec technology/bitstream.

@plinss
Copy link
Member

plinss commented Apr 20, 2020

Given that these APIs offer more control than existing places where codecs are used, such as MediaRecorder, has any thought been given to exposing the existing codes via these APIs? For example, I can see starting a MediaRecorder, getting the video encoder and setting options (or swapping an encoder I construct), rather then extending the MediaRecorder APIs to have similar control surface...

@cynthia
Copy link
Member

cynthia commented Apr 22, 2020

Raised w3c/webcodecs#50 to track the image use case.

@cynthia cynthia added Progress: review complete Resolution: satisfied The TAG is satisfied with this design labels May 26, 2020
@dbaron
Copy link
Member

dbaron commented May 26, 2020

@cynthia and I are looking at this in a breakout at the TAG's virtual face-to-face.

We (the TAG) have looked at this a few times, and I think we're generally pretty happy with it. The design looks appealingly simple -- which is either great (if it can stay simple in the end), or it's a sign that a bunch of things were missed in the design. We're hoping it's the former.

Peter's comment gets at one of the tricky issues with adding new fundamental APIs that other things are built on, when those other things already exist. When that happens, there's generally a process of making the existing features, like MediaRecorder, work with the new fundamental API. That doesn't need to happen all at once, but it does often require somebody making sure that progress happens (e.g. by filing appropriate issues on places that will need to be changed, so we don't lose track) so that the desired end state can be reached. We hope that happens here once the foundation is ready.

So I think we're going to close this issue. Thanks requesting our feedback. Let us know if there's anything else you'd like us to look at later on in the process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Progress: review complete Resolution: satisfied The TAG is satisfied with this design Review type: CG early review An early review of general direction from a Community Group Topic: media Venue: Media WG Venue: WICG
Projects
None yet
Development

No branches or pull requests

7 participants