w3ctag / design-reviews

W3C specs and API reviews
Creative Commons Zero v1.0 Universal
319 stars 55 forks source link

WebRTC-SVC (Scalable Video Coding) #837

Closed dontcallmedom closed 11 months ago

dontcallmedom commented 1 year ago

I'm requesting a TAG re-review of WebRTC Scalable Video Coding.

WebRTC Scalable Video Coding allows to use the capabilities of modern video codecs to combine several video quality into a single encoded stream sent via WebRTC to an endpoint that can then dispatch the right substream to the different parties based on their receiving capabilities.

Further details:

You should also know that there was a previous review of this specification 4 years ago: https://github.com/w3ctag/design-reviews/issues/396

We'd prefer the TAG provide feedback as (please delete all but the desired option):

🐛 open issues in our GitHub repo for each point of feedback

johnathan79717 commented 1 year ago

The Security and Privacy self-review link is invalid. I think you mean https://w3c.github.io/webrtc-svc/#privacy and https://w3c.github.io/webrtc-svc/#security.

Orphis commented 1 year ago

We had a TAG review already done in #396 , is it not sufficient?

aboba commented 1 year ago

@Orphis I had the same question. The only major change in the API from the last TAG review was the use of Media Capabilities API instead of getCapabililties().

dontcallmedom commented 1 year ago

396 is 4 years old; so this request is to check that the intervening changes (in this spec and in the platform in general) don't warrant new feedback

dontcallmedom commented 1 year ago

@torgo any ETA on this review? We're trying to assess when we might be ready to request CR

hober commented 12 months ago

@Orphis I had the same question. The only major change in the API from the last TAG review was the use of Media Capabilities API instead of getCapabililties().

It looks like almost everything in the spec has changed since our last review.

torgo commented 12 months ago

Hi folks -

Thanks for sending us this review. Given what @adoba wrote above concerning the changes and the fact that Media Capabilities API itself enjoys multi-stakeholder support, we'd like to close this review. However, it looks to us like a lot of additional changes were made to the spec. @dontcallmetom can you confirm that this is the only substantive change?

dontcallmedom commented 12 months ago

This isn't the only substantive change, although it's probably the most platform-impacting one.

The list of non-editorial pull requests merged since the last TAG review can be classified as follows:

torgo commented 11 months ago

Hi @dontcallmedom thanks for that clarification. On that basis we're happy to see this move forward. Thanks for bringing to us!