Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: James Almer <jamrial@gmail.com>
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH v3 3/3] tests/fate/mov: add a test for reading and writing amve box
Date: Sun, 4 Feb 2024 09:45:11 -0300
Message-ID: <9190bdc1-bb86-463d-b51c-3e3d40ef544a@gmail.com> (raw)
In-Reply-To: <0101018d73d5eba5-6fbdb71f-6991-43f5-be1e-247fc3187d57-000000@us-west-2.amazonses.com>

On 2/4/2024 8:16 AM, Cosmin Stejerean via ffmpeg-devel wrote:
> From: Cosmin Stejerean <cosmin@cosmin.at>
> 
> ---
>   tests/fate/mov.mak            |  5 +++++
>   tests/ref/fate/mov-read-amve  |  8 ++++++++
>   tests/ref/fate/mov-write-amve | 33 +++++++++++++++++++++++++++++++++
>   3 files changed, 46 insertions(+)
>   create mode 100644 tests/ref/fate/mov-read-amve
>   create mode 100644 tests/ref/fate/mov-write-amve
> 
> diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak
> index f202f36d96..ebad6ff2f4 100644
> --- a/tests/fate/mov.mak
> +++ b/tests/fate/mov.mak
> @@ -9,6 +9,7 @@ FATE_MOV = fate-mov-3elist \
>              fate-mov-frag-encrypted \
>              fate-mov-tenc-only-encrypted \
>              fate-mov-invalid-elst-entry-count \
> +           fate-mov-write-amve \
>              fate-mov-gpmf-remux \
>              fate-mov-440hz-10ms \
>              fate-mov-ibi-elst-starts-b \
> @@ -25,6 +26,7 @@ FATE_MOV_FFPROBE = fate-mov-neg-firstpts-discard \
>                      fate-mov-zombie \
>                      fate-mov-init-nonkeyframe \
>                      fate-mov-displaymatrix \
> +                   fate-mov-read-amve \
>                      fate-mov-spherical-mono \
>                      fate-mov-guess-delay-1 \
>                      fate-mov-guess-delay-2 \
> @@ -109,6 +111,9 @@ fate-mov-init-nonkeyframe: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_packets -
>   
>   fate-mov-displaymatrix: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=display_aspect_ratio,sample_aspect_ratio:stream_side_data_list -select_streams v -v 0 $(TARGET_SAMPLES)/mov/displaymatrix.mov
>   
> +fate-mov-read-amve: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream_side_data_list -select_streams v -v 0 $(TARGET_SAMPLES)/mov/amve.mov

There's no need for this test if you add the other. The writing one 
succeeding means the box was read and propagated from the input file.

> +fate-mov-write-amve: CMD = transcode mov $(TARGET_SAMPLES)/mov/amve.mov mp4 "-c copy" "-c copy -t 0.5" "-show_entries stream_side_data_list"

-c:v

> +
>   fate-mov-spherical-mono: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream_side_data_list -select_streams v -v 0 $(TARGET_SAMPLES)/mov/spherical.mov
>   
>   fate-mov-gpmf-remux: CMD = md5 -i $(TARGET_SAMPLES)/mov/fake-gp-media-with-real-gpmf.mp4 -map 0 -c copy -fflags +bitexact -f mp4
> diff --git a/tests/ref/fate/mov-read-amve b/tests/ref/fate/mov-read-amve
> new file mode 100644
> index 0000000000..91d34d94dd
> --- /dev/null
> +++ b/tests/ref/fate/mov-read-amve
> @@ -0,0 +1,8 @@
> +[STREAM]
> +[SIDE_DATA]
> +side_data_type=Ambient viewing environment
> +ambient_illuminance=3140000/10000
> +ambient_light_x=15635/50000
> +ambient_light_y=16450/50000
> +[/SIDE_DATA]
> +[/STREAM]
> diff --git a/tests/ref/fate/mov-write-amve b/tests/ref/fate/mov-write-amve
> new file mode 100644
> index 0000000000..115cdbd9f0
> --- /dev/null
> +++ b/tests/ref/fate/mov-write-amve
> @@ -0,0 +1,33 @@
> +850c56be1114aa21a2e41bd4ea3da144 *tests/data/fate/mov-write-amve.mp4
> +23677 tests/data/fate/mov-write-amve.mp4
> +#extradata 0:       49, 0x7f8d1145
> +#tb 0: 1/15360
> +#media_type 0: video
> +#codec_id 0: h264
> +#dimensions 0: 640x360
> +#sar 0: 0/1
> +0,      -1024,          0,      512,    11849, 0xf21aa1d0
> +0,       -512,       2048,      512,     1572, 0xf0c41b68, F=0x0
> +0,          0,       1024,      512,      347, 0x9b8daabf, F=0x0
> +0,        512,        512,      512,      195, 0x557e58db, F=0x0
> +0,       1024,       1536,      512,      134, 0x423541b4, F=0x0
> +0,       1536,       3072,      512,     1454, 0xe5a2cdad, F=0x0
> +0,       2048,       2560,      512,      168, 0xd0ef5402, F=0x0
> +0,       2560,       5120,      512,     1395, 0x603eb602, F=0x0
> +0,       3072,       4096,      512,      304, 0x69cc92a6, F=0x0
> +0,       3584,       3584,      512,      145, 0x3f1a4462, F=0x0
> +0,       4096,       4608,      512,      154, 0x953851d1, F=0x0
> +0,       4608,       5632,      512,      876, 0xad65ace7, F=0x0
> +0,       5120,       7680,      512,      742, 0x6b6d689f, F=0x0
> +0,       5632,       6656,      512,      177, 0xa4f2573b, F=0x0
> +0,       6144,       6144,      512,      101, 0xb0722d2b, F=0x0
> +0,       6656,       7168,      512,      105, 0x1f6033ed, F=0x0
> +0,       7168,       9728,      512,      589, 0xcd912063, F=0x0
> +[STREAM]
> +[SIDE_DATA]
> +side_data_type=Ambient viewing environment
> +ambient_illuminance=3140000/10000
> +ambient_light_x=15635/50000
> +ambient_light_y=16450/50000
> +[/SIDE_DATA]
> +[/STREAM]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  reply	other threads:[~2024-02-04 12:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20240204111648.76522-1-cosmin@cosmin.at>
2024-02-04 11:16 ` [FFmpeg-devel] [PATCH v3 0/3] avformat/mov: add support for 'amve' ambient viewing environment box Cosmin Stejerean via ffmpeg-devel
     [not found] ` <20240204111648.76522-2-cosmin@cosmin.at>
2024-02-04 11:16   ` [FFmpeg-devel] [PATCH v3 1/3] avcodec: add ambient viewing environment packet side data Cosmin Stejerean via ffmpeg-devel
2024-02-04 11:45     ` Anton Khirnov
     [not found]       ` <57A56D7C-E692-4A97-93AC-8436B5B38739@cosmin.at>
2024-02-04 11:51         ` Cosmin Stejerean via ffmpeg-devel
     [not found] ` <20240204111648.76522-3-cosmin@cosmin.at>
2024-02-04 11:16   ` [FFmpeg-devel] [PATCH v3 2/3] avformat/mov: add support for 'amve' ambient viewing environment box. As defined in ISOBMFF (ISO/IEC 14496-12) document Cosmin Stejerean via ffmpeg-devel
2024-02-04 12:28     ` James Almer
     [not found] ` <20240204111648.76522-4-cosmin@cosmin.at>
2024-02-04 11:16   ` [FFmpeg-devel] [PATCH v3 3/3] tests/fate/mov: add a test for reading and writing amve box Cosmin Stejerean via ffmpeg-devel
2024-02-04 12:45     ` James Almer [this message]
     [not found]       ` <C603FBA4-EA08-4952-A2D3-82C5EC5A1E04@cosmin.at>
2024-02-04 13:01         ` Cosmin Stejerean via ffmpeg-devel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9190bdc1-bb86-463d-b51c-3e3d40ef544a@gmail.com \
    --to=jamrial@gmail.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git