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".
next prev parent 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