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.

Sign up for GitHub

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

Jump to bottom

Description of CSP_UNKNOWN mentions the "source video transfer function", which seems wrong #309

Open
wantehchang opened this issue Aug 24, 2020 · 1 comment · May be fixed by #355
Open

Description of CSP_UNKNOWN mentions the "source video transfer function", which seems wrong #309

wantehchang opened this issue Aug 24, 2020 · 1 comment · May be fixed by #355

Comments

@wantehchang
Copy link
Collaborator

wantehchang commented Aug 24, 2020 •
edited
Loading

[I originally filed this issue in https://crbug.com/aomedia/2793.]

What version / commit were you testing with?
(git describe can produce this info.)

AV1 Bitstream & Decoding Process Specification
Last modified: 2019-01-08 11:48 PT
Version 1.0.0 with Errata 1

In Section 6.4.2. Color config semantics, the last table describes the chroma sample position name CSP_UNKNOWN as:

Unknown (in this case the source video transfer function must be signaled outside the AV1 bitstream)

It is not clear why the "source video transfer function" is mentioned. Should that be changed to "chroma sample position"?
@xzhaao
Copy link

xzhaao commented Oct 5, 2020

In addition, it is not clear why this spec specifies some behaviors outside the AV1 bitstream, it seems outside the scope of this spec.

wantehchang added a commit to wantehchang/av1-spec that referenced this issue Jul 2, 2024
@wantehchang
Fix a typo in the description of CSP_UNKNOWN
44fc2f0 
Change "source video transfer function" to "chroma sample position" in
the description of CSP_UNKNOWN.

Fixes AOMediaCodec#309.
@wantehchang wantehchang linked a pull request Jul 2, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a typo in the description of CSP_UNKNOWN wantehchang/av1-spec
2 participants
@wantehchang @xzhaao