MPEGGroup / FileFormat

MPEG file format discussions
20 stars 0 forks source link

Clause A.7 (Extractors for HEVC and L-HEVC tracks) - editorial improvements #55

Closed fmaze closed 1 week ago

fmaze commented 2 years ago

Issue 1: Clause A.7.1 states:

An extractor contains one or more constructors. The following constructors are specified:
a)  A sample constructor extracts, by reference, NAL unit data from a sample of another track.
b)  An in-line constructor includes NAL unit data.

The list of possible constructors is incomplete. It could be replaced as follows:

An extractor contains one or more constructors. The following constructors are specified:
a)  A sample constructor extracts, by reference, NAL unit data from a sample of another track.
b)  An in-line constructor includes NAL unit data.
c)  A sample constructor extracts, by reference, NAL unit data from a sample of a track selected from a track group.
d)  A reference constructor allows including a default constructor by explicit reference in an extractor.
e)  A default reference constructor allows including a default constructor by implicit reference in an extractor.
f)  An in-line constructor includes beginning of a NAL unit data without any NALUnitLength field.

Issue 2: about clause A.7.6.2 on semantics of Sample constructor from a track group: The semantics of some fields are missing. I suggest to add the following:

sample_offset: as specified in  A.7.4.1.2.
data_length: as specified in A.7.4.1.2.
nalu_idx_field_size: specifies the field size of nalu_idx. If the value is 0, the field size is 8 bits, otherwise the field size is 16 bits.
yekuiwang commented 1 year ago

I will handle this.

yekuiwang commented 1 year ago

Fixed in WG03N0801_22325_14496_15_Ed6_draft_DAM2_text (Draft text of ISO/IEC 14496-15 6th edition DAM 2 Picture-in-picture support and other extensions). However, note that there is no field of data_length in the syntax, thus that part is not added.

cconcolato commented 1 week ago

@fmaze can we close this issue or do we need to do anything about data_length?

fmaze commented 1 week ago

There is no more issue. we can close it.