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

Sentences about placement for show_frame = 0 #57

Open
cconcolato opened this issue May 30, 2023 · 1 comment
Open

Sentences about placement for show_frame = 0 #57

cconcolato opened this issue May 30, 2023 · 1 comment
Assignees
Labels
editorial For issues that are purely editorial specification For issues or questions around the specification itself

Comments

@cconcolato
Copy link
Collaborator

The sentence:

HDR10+ Metadata OBUs are not provided when show_frame = 0.

is a consequence of the sentences in the previous paragraph, because:

  • a show_frame = 0 cannot be on its own in a Temporal unit
  • if the show_frame = 0 had a HDR10+ Metadata OBU, either the TU would have 2 HDR10+ Metadata OBU and that is invalid, or the show_frame = 1 in the same TU would not have an HDR10+ Metadata OBU, and this is also invalid.

We should:

  • remove the assert so that it does not appear in the conformance report
  • consider making the sentence a note to make it clear that it is not an additional statement

The second sentence:

For non-layered streams, there is only one HDR10+ Metadata OBU per temporal unit.

should also not be wrapped into an assert and maybe put into a NOTE. It's a rephrase/consequence of the previous paragraph.

@cconcolato cconcolato added specification For issues or questions around the specification itself editorial For issues that are purely editorial labels Jun 19, 2023
podborski added a commit that referenced this issue Sep 11, 2023
@cconcolato
Copy link
Collaborator Author

@podborski I think this issue can be closed, correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial For issues that are purely editorial specification For issues or questions around the specification itself
Projects
None yet
Development

No branches or pull requests

2 participants