Skip to content

Add requirements to VideoFrame metadata registry definition #597

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

Merged
merged 3 commits into from
Nov 15, 2022

Conversation

chrisn
Copy link
Member

@chrisn chrisn commented Oct 21, 2022

Following the discussion in #559, this PR covers further changes to adding, changing, and deprecating registry entries.

Covers adding, changing, and deprecating registry entries
@chrisn chrisn requested a review from aboba October 21, 2022 17:27
Copy link
Collaborator

@aboba aboba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 40-42: You might say, "The specification defining the metadata entry must provide clearly defined semantics." That is, the semantics should be defined in the specification, not the registration request.

@chrisn
Copy link
Member Author

chrisn commented Oct 24, 2022

Thanks, I have pushed an update to clarify this.

@chrisn chrisn requested review from youennf, padenot and tidoust October 25, 2022 09:02
Comment on lines +38 to +39
3. Each metadata entry must be defined by a W3C specification that has
reached consensus in the originating Working Group.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Three questions:

  1. Does this need to be restricted to W3C specifications?
  2. Should this be restricted to normative specifications?
  3. What does it mean for a W3C specification to have "reached consensus in the originitaing Working Group"?

For 1., I realize that it's hard to come up with an explicit set of requirements for a specification to be considered good enough. A document that comes to mind are the considerations the Director takes into account when evaluating normative references from W3C documents. These considerations include considerations such as:

For 2., I'm wondering whether the group is ok with specifications on the Note track, as done for codec registrations but perhaps not such a good example to follow?

For 3., a status on the Recommendation track (e.g. Candidate Recommendation), would make things clearer. Also, I'm thinking that the group might want to add entries even when a spec is still under development?

I'm having a hard time coming up with a proposal that is not convoluted, so perhaps it's fine to leave this a bit under specified. If the group wanted to open the registry to non-W3C specs, the following wording could be considered:

Suggested change
3. Each metadata entry must be defined by a W3C specification that has
reached consensus in the originating Working Group.
3. Each metadata entry must be defined in an established standard with licensing terms that are consistent with royalty-free implementation of W3C Recommendations. The Media Working Group will evaluate candidate specifications following [considerations used when evaluating normative references from W3C specifications](https://www.w3.org/2013/09/normative-references) [[NORMATIVE-REFERENCES]].

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tidoust, my thoughts on your questions:

  1. We discussed limiting to W3C specifications in [WebCodecs VideoFrame metadata registry] Introduce VideoFrame metadata #559 (comment), my interpretation is partly as a way to ensure compatible licensing
  2. My understanding is that metadata entries should be normative, but they may be optional to implement, per [WebCodecs VideoFrame metadata registry] Introduce VideoFrame metadata #559 (comment)
  3. I expect that we will want to add entries in draft state. I saw consensus here as: Media WG having consensus to add the registration, and the originating WG having a draft spec to add and consensus on its purpose and scope, it's OK if details are still in development ([WebCodecs VideoFrame metadata registry] Introduce VideoFrame metadata #559 (review))

Copy link
Contributor

@youennf youennf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
With regards to the process, I think keeping it a bit unspecified until we exercise it seems ok.
In general, I would think that metadata entries being normative, they should follow the REC track.
The restriction to W3C specifications (but they might refer to non W3c specs themselves) is fine for now as well, given spec authors will anyway need to interact deeply with the Media WG.

@dalecurtis
Copy link
Contributor

Are we ready to merge this one?

@chrisn
Copy link
Member Author

chrisn commented Oct 28, 2022

I think so, @padenot is this OK from your perspective?

@chrisn
Copy link
Member Author

chrisn commented Nov 15, 2022

@padenot I'd like to merge this, as we now have incoming proposals to add to the registry. Thanks!

@padenot
Copy link
Collaborator

padenot commented Nov 15, 2022

Sure, sorry for missing this, go ahead!

@chrisn chrisn merged commit 71b75a6 into main Nov 15, 2022
@chrisn chrisn deleted the metadata-registry-policy branch November 15, 2022 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants