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

Clarify normative aspects of TS header and AF fields #16

Open
cconcolato opened this issue Sep 28, 2022 · 4 comments
Open

Clarify normative aspects of TS header and AF fields #16

cconcolato opened this issue Sep 28, 2022 · 4 comments

Comments

@cconcolato
Copy link
Collaborator

The DVB draft specification for AV1 mentions that:

  • payload_unit_start_indicator shall be set to 1
  • adaptation_field_control bits shall be set to "11"
  • The random_access_indicator shall be set to 1 when the PES packet contains an AV1 KeyFrame
  • The elementary_stream_priority_indicator and intra frame interact

It's unclear why these statements are not in the AV1-MPEG-2 TS specification. Could these be different outside of DVB environments?
In any cases, the specification should talk about these fields: either to mandate the behavior or to indicate that the behavior can vary depending on application environments.

@cconcolato
Copy link
Collaborator Author

cconcolato commented Oct 3, 2022

Current DVB text is:
image
and
image

@tbiat
Copy link
Collaborator

tbiat commented Oct 3, 2022

payload_ ==> It's implicit and obvious.
adaptation_field_... ==> dvb mandates the adaptation field for random access AV1 DVB_RAP, we might not need it for other TS-based applications.
Decision : share the DVB spec part related to this with the group. Revisit next week.

@tbiat
Copy link
Collaborator

tbiat commented Oct 17, 2022

We don't want RAP in this spec to conflict with AV1 DVB-RAP.
Put a note : recommended practice / 'should contain'.
Decision : take offline with Cyril. Propose a wording by email for approval.

@cconcolato
Copy link
Collaborator Author

The spec says:

A PES shall encapsulate one, and only one, AV1 access unit as defined in clause 4.3.
We could add a note that says "as a consequence, the PUSI is set to 1"

The spec does neither mention 'adaptation_field_control' nor 'random_access_indicator', not even "random". We should at least have: "When the random access indicator is set to 1, the AV1 access unit shall contain an AV1 Key Frame".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants