Re: [Cellar] Colour Format proposal

Hendrik Leppkes <[email protected]> Thu, 14 Jan 2016 13:11:42 +0100
Newsgroups gmane.comp.multimedia.matroska.devel
Message-ID <CA+anqdz1-1qP+=11AkwaEq097twP2AMH1tFwa31V0kcFC81Myw@mail.gmail.com>
On Thu, Jan 14, 2016 at 1:04 PM, Steve Lhomme <[email protected]> wrote:
> 2016-01-06 23:30 GMT+01:00 Frank Galligan <[email protected]>:
>>
>>
>> On Tue, Jan 5, 2016 at 1:58 PM, Jerome Martinez <[email protected]>
>> wrote:
>>>
>>> Thanks for your detailed proposal.
>>> It is a lot of items, I am not expert in all domains but I have some
>>> comments:
>>
>> Neither am I, but I figured we might as well start from a proposal and
>> iterate from there.
>>
>>>
>>> Name: MatrixCoefficients (or Matrix. You don't use "Colour" with
>>> "Primaries" element, so I would not use it here too)
>>
>>  Sounds good to me.
>>
>>>
>>>
>>>
>>>
>>>
>>> Element Name: BitsPerChannel
>>> Level:        5
>>> ID:           [55][A2]
>>> Mandatory:    ma
>>> Multiple:     -
>>> Default:      8
>>> Type:         u
>>> Description:  Number of bits per channel. This number may be less for
>>> specific
>>>               channels depending on the ColourFormat and
>>> ChromaSubsampling.
>>>
>>>
>>>
>>> BitsPerChannel is misleading, used to be the technical count of bits.
>>> I propose QuantizationBits or ValidBitsPerChannel
>>> Not mandatory, not default (this value is often unknown)
>>
>> When authoring the video, I'm not sure why the software would not know the
>> bits per channel. Can you describe a use case where the this is unknown?
>
> When remuxing an older file. Either from Matroska source or AVI or other.
>
> Now if elements are mandatory with a default value to "unknown" it's fine.
>

Whats the point of a mandatory element with an "unknown" default? Just
make it optional then?

- Hendrik
_______________________________________________
Matroska-devel mailing list
[email protected]
http://lists.matroska.org/cgi-bin/mailman/listinfo/matroska-devel
Read Matroska-Devel on GMane: http://dir.gmane.org/gmane.comp.multimedia.matroska.devel