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

Add to codec registry requirements #693

Merged
merged 2 commits into from
Jul 6, 2023
Merged

Conversation

chrisn
Copy link
Member

@chrisn chrisn commented Jun 23, 2023

This is intended to resolve #426.

  • Added a requirement for a public specification that is stable
  • Added that the WG may consult external expertise as part of its review (but not go so far as producing a written review with a recommendation, as WebCodecs registration policy #426 suggested)
  • Fixes the numbering in the source document

@aboba for review

Changes:

- Adds a requirement for a public specification that is stable
- The WG may consult external expertise as part of its review
- Fixes the numbering in the source document
@chrisn chrisn added the registry pertains to new or updated registry entry label Jun 23, 2023
@chrisn chrisn requested a review from aboba June 23, 2023 16:35
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.

Line 48: What does "stable" mean in this context? Taking the IETF as an example, would an Internet-Draft be considered stable upon WG adoption? Upon approval for publication as an RFC?

@chrisn
Copy link
Member Author

chrisn commented Jul 4, 2023

This gives a pretty good set of considerations: https://www.w3.org/2013/09/normative-references#stability

But, adding new registrations is pretty infrequent so I'm wondering whether we really need say anything, and leave to the WG evaluation at the time.

@aboba
Copy link
Collaborator

aboba commented Jul 5, 2023

@chrisn I think we should leave it to WG evaluation.

Note that there are other requirements in the normative reference gujde that codec specifications may not be able to meet. For example, the specification for EVC (,"ISO/IEC 23094-1 Essential Video Coding") is behind a paywall at https://www.iso.org/standard/57797.html In IETF, the typical requirement is only that versions of the codec specification can be made available to reviewers upon request.

@chrisn
Copy link
Member Author

chrisn commented Jul 5, 2023

To allow for paywalled specs we could change the wording from "must include ... a link to a public specification" to "must include ... a reference to the codec's specification" - and we'd similarly want the spec to be made available for review (we have done this from time to time in MEIG).

@aboba aboba merged commit c414b9c into main Jul 6, 2023
17 checks passed
@aboba aboba deleted the chrisn-registry-requirements branch July 6, 2023 16:10
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jul 6, 2023
SHA: c414b9c
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
registry pertains to new or updated registry entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebCodecs registration policy
2 participants