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