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