[FFmpeg-devel] [NUT-devel] [PATCH] Add st_sd_displaymatrix for storing the stream display matrix

[Michael Niedermayer]

On Fri, Nov 20, 2020 at 12:28:59PM +0100, Matthias Neugebauer wrote:
> See FFmpeg patch: https://patchwork.ffmpeg.org/project/ffmpeg/patch/F7848CEE-46C1-4CAF-9191-A7BABF5D61B4@mailbox.org/
> ---
> docs/nut.txt | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/docs/nut.txt b/docs/nut.txt
> index 3641c01..ea49eba 100644
> --- a/docs/nut.txt
> +++ b/docs/nut.txt
> @@ -1,5 +1,5 @@
> ==================================
> -NUT Open Container Format 20201104
> +NUT Open Container Format 20201120
> ==================================
> 
> 

This doesnt apply as the whitespaces seem to have been mangled by your MUA
or some other part of the mail pipeline



> @@ -1045,6 +1045,12 @@ info packet types
>          framerates in the stream). Note, this value is just a guess!
>          For example, if the time base is 1/90000 and all frames have either
>          approximately 3600 or 1800 timer ticks, then r_frame_rate will be 50/1.
> +    "st_sd_displaymatrix"
> +        Display matrix of the stream.
> +        This stores the 3x3 display matrix of the stream as nine unsigned
> +        integers seperated by colons. The display matrix is a side data of a
> +        video stream and necessary for displaying the stream correctly, e.g.
> +        rotating it.

ok but this needs to be a bit more precisse
is a 
1 2 3
4 5 6
7 8 9
matrix stored as
1 2 3 4 5 6 7 8 9 or 1 4 7 2 5 8 3 6 9 or something else
also this needs a clear definition what a display matrix is or a 
link / reference to some other specification which clearly defines what a
display matrix is

Without this there could be interoperability issues if people interpret
this in different ways

iam CC-ing ffmpeg-devel as i suspect there are more interrested people there
to comment than on nut-devel and the subject also touches the ffmpeg 
implementation

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20201213/f9658fb5/attachment.sig>