search

w3c / webrtc-svc

W3C Scalable Video Coding (SVC) Extension for WebRTC
https://w3c.github.io/webrtc-svc/
Other
39 stars 14 forks source link

Why is scalabilityMode not an enum? #87

Closed henbos closed 1 year ago

henbos commented 1 year ago

According to the guidelines, any proposed string will be added to the table. A set of valid strings sounds like an enum to me, even if that enum type is later amended it's still an enum.

henbos commented 1 year ago

AFAIK this is not observably different from JS and is an editorial issue, so feel free to close as "it doesn't matter", but the value is exposed in a few different places (getParameters, getStats) and handled as a raw string in the code

aboba commented 1 year ago

This issue is a duplicate of Issue https://github.com/w3c/webrtc-svc/issues/42 which was raised by the TAG.

The issue was discussed at the 2020 TPAC meeting on Thursday, October 22, 2020. The slides are here: https://docs.google.com/presentation/d/1WHocMl47fukck4YQ6RuPzfagAj6bW0SjUELzwguC3qs/edit#slide=id.g9be42c5e5c_0_220

The consensus at the meeting was to keep scalabilityMode as a DOMString. Video of the meeting is here: https://www.youtube.com/watch?v=gZsZIwfvw28

Dom's summary of the meeting discussion is at the end of the thread on Issue https://github.com/w3c/webrtc-svc/issues/42