-
Notifications
You must be signed in to change notification settings - Fork 136
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
Inconsistent SVC metadata between WebCodecs and WebRTC Encoded Transform API #619
Inconsistent SVC metadata between WebCodecs and WebRTC Encoded Transform API #619
Comments
+1 for consistency, but defer to @Djuffin |
It's unusual to add fields unless there is at least one implementation that makes use of them. But in this case compatibility with Encoded Transforms seems like a pretty good reason to add these fields. I think we can even rename |
The Encoded Transform metadata has now been updated. However, it still is not completely compatible with the WebCodecs extended SVC metadata proposal. |
A summary of where we are now. Here is the Here is what is in WebCodecs Section 6.7: Here is what was in the extended WebCodecs SVC metadata proposal: |
@pthatcherg @alvestrand Can you take a look? |
Minutes from 11 Apr 2023 Media WG meeting: https://www.w3.org/2023/04/11-mediawg-minutes.html#t03 |
Currently the
EncodedChunkMetadata
dictionary includesSvcOutputMetadata
, which only supports temporal scalability (temporalLayerId).However, within the Encoded-Transform API, RTCEncodedFrameMetadata includes its own (incompatible) SVC metadata.
Here is a comparison of the two approaches:
https://docs.google.com/presentation/d/1lFAUSvApbBYfBNJH_xcRW0YjD0aF5T1ZqjyyDJMesJw/edit#slide=id.g1a4ac56601a_3_0
Can we define the complete set of SVC metadata in WebCodecs and then have it inherited by Encoded Transform?
Related: w3c/webrtc-encoded-transform#170
The text was updated successfully, but these errors were encountered: