From: Steven Liu <lingjiujianke@gmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH v3 1/3] tests/fate/flvenc: add hevc in enhanced flv test case Date: Thu, 20 Jul 2023 16:45:53 +0800 Message-ID: <CADxeRw=rYhRSyuKJVSVsKdwZJLr4WFvOEBV=hTBdVovx2fiH6w@mail.gmail.com> (raw) In-Reply-To: <GV1P250MB073781048E8C45FD312BE10D8F3EA@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> Andreas Rheinhardt <andreas.rheinhardt@outlook.com> 于2023年7月20日周四 16:41写道: > > Steven Liu: > > Andreas Rheinhardt <andreas.rheinhardt@outlook.com> 于2023年7月19日周三 18:32写道: > >> > >> Steven Liu: > >>> Signed-off-by: Steven Liu <lq@chinaffmpeg.org> > >>> --- > >>> tests/fate/flvenc.mak | 7 ++++++- > >>> tests/ref/fate/enhanced-flv-hevc | 8 ++++++++ > >>> 2 files changed, 14 insertions(+), 1 deletion(-) > >>> create mode 100644 tests/ref/fate/enhanced-flv-hevc > >>> > >>> diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak > >>> index 5032fface3..6b2fbe8c89 100644 > >>> --- a/tests/fate/flvenc.mak > >>> +++ b/tests/fate/flvenc.mak > >>> @@ -1,5 +1,10 @@ > >>> FATE_FLVENC_FFMPEG_FFPROBE-$(call TRANSCODE, FLV, FLV, RAWVIDEO_DECODER SCALE_FILTER TESTSRC_FILTER LAVFI_INDEV) += fate-flv-add_keyframe_index > >>> fate-flv-add_keyframe_index: CMD = transcode "lavfi -graph testsrc=r=7:n=2:d=20" "foo" flv "-vf scale -c:v flv1 -dct int -g 7 -flvflags add_keyframe_index" "-c copy -t 0.1" "-show_entries format_tags" > >>> > >>> +FATE_ENHANCED_FLVENC_FFMPEG-$(call REMUX, MOV, FLV_MUXER) += fate-enhanced-flv-hevc > >>> +fate-enhanced-flv-hevc: CMD = transcode mov $(TARGET_SAMPLES)/hevc/dv84.mov\ > >>> + flv "-vcodec copy -map 0:v -frames 1" > >>> + > >>> FATE_FFMPEG_FFPROBE += $(FATE_FLVENC_FFMPEG_FFPROBE-yes) > >>> -fate-flvenc: $(FATE_FLVENC_FFMPEG_FFPROBE-yes) > >>> +FATE_SAMPLES_FFMPEG += $(FATE_ENHANCED_FLVENC_FFMPEG-yes) > >>> +fate-flvenc: $(FATE_FLVENC_FFMPEG_FFPROBE-yes) $(FATE_ENHANCED_FLVENC_FFMPEG-yes) > >>> diff --git a/tests/ref/fate/enhanced-flv-hevc b/tests/ref/fate/enhanced-flv-hevc > >>> new file mode 100644 > >>> index 0000000000..41eaefaf6e > >>> --- /dev/null > >>> +++ b/tests/ref/fate/enhanced-flv-hevc > >>> @@ -0,0 +1,8 @@ > >>> +afed0105d121688bad7dc94218256571 *tests/data/fate/enhanced-flv-hevc.flv > >>> +64396 tests/data/fate/enhanced-flv-hevc.flv > >>> +#tb 0: 1/30 > >>> +#media_type 0: video > >>> +#codec_id 0: rawvideo > >>> +#dimensions 0: 1920x1080 > >>> +#sar 0: 0/1 > >>> +0, 0, 0, 1, 6220800, 0x1f259c93 > >> > >> Why are you restricting this to one frame? And why are you not copying > >> the audio, too (this would test multiplexing)? After all, FLV supports AAC. > >> Furthermore, you are copying the video when creating the intermediate > >> FLV file, yet then you are decoding it at the next stage. This decoding > >> would necessitate a dependency on the HEVC decoder (and maybe the parser > >> as well as the extract_extradata bsf?). Alternatively, one can just use > >> codec copy when reading the intermediate file as well (the latter is my > >> preferred approach). > >> The same remarks also apply to the other patches. > > Hi Andreas, > > > > > > What about do it like this: > > > > FATE_ENHANCED_FLVENC_FFMPEG-$(call TRANSCODE, RAWVIDEO HEVC, FLV MOV, > > HEVCPARSE) += fate-enhanced-flv-hevc > > fate-enhanced-flv-hevc: CMD = transcode mov $(TARGET_SAMPLES)/hevc/dv84.mov\ > > flv "-c copy" "-af aresample" > > > > You would not need to resample when you simply copied the stuff instead > of decoding. It will get error message if i don't use resample: ffmpeg version N-111514-g6efbbe7d25 Copyright (c) 2000-2023 the FFmpeg developers built with Apple clang version 14.0.3 (clang-1403.0.22.14.1) configuration: --prefix=/usr/local/ --libdir=/usr/local/lib/ --cc=clang --enable-htmlpages --enable-libx264 --enable-gpl --extra-cflags='-I/usr/local/include/GraphicsMagick/ -I/usr/local/opt/openssl/include/ -O0 -g3 -fsanitize=address -Wno-error -fPIC -I/usr/local/include -fno-omit-frame-pointer' --extra-ldflags='-O0 -g3 -fsanitize=address -fno-omit-frame-pointer -Wno-error -fPIC -L/usr/local/lib -L/usr/local/opt/openssl/lib -lpng' --enable-libfreetype --enable-fontconfig --enable-libspeex --enable-libopus --enable-libzmq --enable-libx265 --enable-libass --enable-videotoolbox --disable-optimizations --enable-audiotoolbox --enable-opengl --disable-stripping --samples=../../fate-suite/ --enable-encoder=hevc_videotoolbox --enable-hwaccel=hevc_videotoolbox --enable-hwaccel=h264_videotoolbox --enable-openssl --enable-nonfree --disable-shared --enable-libfdk_aac --enable-libxml2 --enable-libmp3lame --enable-libaom --enable-lcms2 --enable-libwebp --enable-libvpx libavutil 58. 14.100 / 58. 14.100 libavcodec 60. 22.100 / 60. 22.100 libavformat 60. 10.100 / 60. 10.100 libavdevice 60. 2.101 / 60. 2.101 libavfilter 9. 8.102 / 9. 8.102 libswscale 7. 3.100 / 7. 3.100 libswresample 4. 11.100 / 4. 11.100 libpostproc 57. 2.100 / 57. 2.100 Input #0, flv, from '/Users/liuqi/multimedia/upstream_ffmpeg/ufbuild/tests/data/fate/enhanced-flv-hevc.flv': Metadata: major_brand : qt minor_version : 0 compatible_brands: qt com.apple.quicktime.creationdate: 2021-05-29T15:14:19-0400 com.apple.quicktime.make: Apple com.apple.quicktime.model: iPhone 12 mini com.apple.quicktime.software: 14.4.1 Duration: 00:00:03.40, start: 0.000000, bitrate: 8469 kb/s Stream #0:0: Video: hevc (Main 10), yuv420p10le(tv, bt2020nc/bt2020/arib-std-b67), 1920x1080, 8479 kb/s, 30 fps, 29.97 tbr, 1k tbn Stream #0:1: Audio: aac (LC), 44100 Hz, stereo, fltp, 148 kb/s Stream mapping: Stream #0:0 -> #0:0 (hevc (native) -> rawvideo (native)) Stream #0:1 -> #0:1 (aac (native) -> pcm_s16le (native)) The filters 'Parsed_anull_0' and 'format_out_0_1' do not have a common format and automatic conversion is disabled. [af#0:1 @ 0x61000000b740] Error reinitializing filters! Failed to inject frame into filter network: Invalid argument Invalid argument Conversion failed! threads=1 > > - Andreas > > _______________________________________________ > 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". _______________________________________________ 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:[~2023-07-20 8:46 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-07-18 7:32 [FFmpeg-devel] [PATCH v1 0/3] Add test cases of hevc, vp9, av1 in enhanced flv Steven Liu 2023-07-18 7:32 ` [FFmpeg-devel] [PATCH v1 1/3] tests/fate/flvenc: add hevc in enhanced flv test case Steven Liu 2023-07-18 9:50 ` Anton Khirnov 2023-07-18 9:54 ` Steven Liu 2023-07-18 11:21 ` [FFmpeg-devel] [PATCH v2 0/3] Add test cases of hevc, vp9, av1 in enhanced flv Steven Liu 2023-07-18 11:21 ` [FFmpeg-devel] [PATCH v2 1/3] tests/fate/flvenc: add hevc in enhanced flv test case Steven Liu 2023-07-18 11:21 ` [FFmpeg-devel] [PATCH v2 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-18 15:46 ` Michael Niedermayer 2023-07-19 9:10 ` Steven Liu 2023-07-18 11:21 ` [FFmpeg-devel] [PATCH v2 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-18 11:21 ` Steven Liu 2023-07-18 11:36 ` [FFmpeg-devel] [PATCH v1 1/3] tests/fate/flvenc: add hevc " Andreas Rheinhardt 2023-07-19 9:03 ` [FFmpeg-devel] [PATCH v3 0/3] Add test cases of hevc, vp9, av1 in enhanced flv Steven Liu 2023-07-19 9:03 ` [FFmpeg-devel] [PATCH v3 1/3] tests/fate/flvenc: add hevc in enhanced flv test case Steven Liu 2023-07-19 10:33 ` Andreas Rheinhardt 2023-07-20 2:07 ` Steven Liu 2023-07-20 8:42 ` Andreas Rheinhardt 2023-07-20 8:45 ` Steven Liu [this message] 2023-07-20 10:06 ` Andreas Rheinhardt 2023-07-20 10:59 ` [FFmpeg-devel] [PATCH v4 0/3] Add test cases of hevc, vp9, av1 in enhanced flv Steven Liu 2023-07-20 11:00 ` [FFmpeg-devel] [PATCH v4 1/3] tests/fate/flvenc: add hevc in enhanced flv test case Steven Liu 2023-07-20 11:00 ` [FFmpeg-devel] [PATCH v4 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-20 11:00 ` [FFmpeg-devel] [PATCH v4 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-20 21:51 ` Michael Niedermayer 2023-07-21 7:54 ` Steven Liu 2023-07-21 8:14 ` Steven Liu 2023-07-24 2:14 ` [FFmpeg-devel] [PATCH v5 1/3] tests/fate/flvenc: add hevc " Steven Liu 2023-07-24 2:14 ` [FFmpeg-devel] [PATCH v5 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-24 2:14 ` [FFmpeg-devel] [PATCH v5 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-24 7:37 ` [FFmpeg-devel] [PATCH v5 1/3] tests/fate/flvenc: add hevc " Andreas Rheinhardt 2023-07-24 7:38 ` Steven Liu 2023-07-24 7:54 ` Andreas Rheinhardt 2023-07-24 9:20 ` [FFmpeg-devel] [PATCH v6 " Steven Liu 2023-07-24 9:20 ` [FFmpeg-devel] [PATCH v6 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-24 9:20 ` [FFmpeg-devel] [PATCH v6 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-08-04 9:59 ` [FFmpeg-devel] [PATCH v6 1/3] tests/fate/flvenc: add hevc " Steven Liu 2023-08-14 6:11 ` [FFmpeg-devel] [PATCH v7 " Steven Liu 2023-08-14 6:11 ` [FFmpeg-devel] [PATCH v7 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-08-14 6:11 ` [FFmpeg-devel] [PATCH v7 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-24 9:22 ` [FFmpeg-devel] [PATCH v5 1/3] tests/fate/flvenc: add hevc " Steven Liu 2023-07-19 9:03 ` [FFmpeg-devel] [PATCH v3 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-19 9:03 ` [FFmpeg-devel] [PATCH v3 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-19 9:11 ` [FFmpeg-devel] [PATCH v1 1/3] tests/fate/flvenc: add hevc " Steven Liu 2023-07-18 7:32 ` [FFmpeg-devel] [PATCH v1 2/3] tests/fate/flvenc: add vp9 " Steven Liu 2023-07-18 7:32 ` [FFmpeg-devel] [PATCH v1 3/3] tests/fate/flvenc: add av1 " Steven Liu 2023-07-18 7:58 ` Steven Liu
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='CADxeRw=rYhRSyuKJVSVsKdwZJLr4WFvOEBV=hTBdVovx2fiH6w@mail.gmail.com' \ --to=lingjiujianke@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