search

ietf-wg-cellar / matroska-specification

Matroska specification.
http://ietf-wg-cellar.github.io/matroska-specification
Other
121 stars 45 forks source link

Colour Space values #337

Open gibix opened 5 years ago

gibix commented 5 years ago

I want to parse \Segment\Tracks\TrackEntry\Video\ColourSpace and is not clear to me witch are all the possible values to handle by a complete mkv demuxer.

dericed commented 5 years ago

I'm interested in making the description more specific to recommend an external authority or to possible list a set as should-be-supported; however at the moment I think the intent is that this field is intended to store the fourcc for uncompressed video as it would be in avi and I'm not sure if there is a decent authority list of those fourccs to cite.

lu-zero commented 5 years ago

The AVI fourccs would clash with the Colour description, I'd deprecate the use (since the current description of the field does not even say that the AVI ones are supported).

Talking with @robUx4 I'll prepare a pull request to have a mapping for:

Expressed as 'M','0', '0', {numeric value matching the list order}.

The Colour should contain all the other useful information.

kodawah commented 5 years ago

I think ColourSpace is poorly named and mostly refers to how data is packed when it is RAW: for example it defines VYUY, and 2YUY which are packing modes, not related to color systems (which is what @lu-zero is referring to) or color spaces (which is what @dericed is referring to i think)

having a ColorSystem entity would be nice, I'm unsure of the usefulness of this ColourSpace entity

robUx4 commented 3 years ago

548 should not close this, the clarifications should be done properly in the codec spec, not the main format spec. It's specific to one codec: V_UNCOMPRESSED.