Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH v1 1/3] tests/fate/flvenc: add hevc in enhanced flv test case
Date: Tue, 18 Jul 2023 13:36:18 +0200
Message-ID: <AS8P250MB074476AA5BBB4D1EEF1AEED88F38A@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <20230718073214.14415-2-lq@chinaffmpeg.org>

Steven Liu:
> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
> ---
>  tests/fate/flvenc.mak            | 11 +++++++++++
>  tests/ref/fate/enhanced-flv-hevc |  6 ++++++
>  2 files changed, 17 insertions(+)
>  create mode 100644 tests/ref/fate/enhanced-flv-hevc
> 
> diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak
> index 5032fface3..a0af45e1a8 100644
> --- a/tests/fate/flvenc.mak
> +++ b/tests/fate/flvenc.mak
> @@ -1,5 +1,16 @@
>  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"
>  
> +
> +tests/data/enhanced_flv_hevc.flv: TAG = GEN
> +tests/data/enhanced_flv_hevc.flv: ffmpeg$(PROGSSUF)$(EXESUF) | tests/data
> +	$(M)$(TARGET_EXEC) $(TARGET_PATH)/$< -nostdin \
> +			-f lavfi -i "testsrc=s=176x144:r=1:n=1:d=0.04" -pix_fmt yuv420p -c:v libx265 -bf 0 -threads 1 -t 0.04 -y $(TARGET_PATH)/tests/data/enhanced_flv_hevc.flv 2>/dev/null
> +

This will encode using libx265 which is beyond our control (and has
issues with Valgrind). This would only work if the codec were lossless;
but your command line isn't.

Anyway, you should instead just use any of our hevc samples and remux it
to flv and read the resulting file. There are lots of examples of how to
do this that also avoid keeping the intermediate file around.

The other two patches have the same issues.

> +FATE_FLVENC_FFMPEG_FFPROBE-$(call TRANSCODE, LIBX265 HEVC, FLV, TESTSRC_FILTER LAVFI_INDEV) += fate-enhanced-flv-hevc
> +fate-enhanced-flv-hevc: tests/data/enhanced_flv_hevc.flv
> +fate-enhanced-flv-hevc: SRC = $(TARGET_PATH)/tests/data/enhanced_flv_hevc.flv
> +fate-enhanced-flv-hevc: CMD = framecrc -auto_conversion_filters -flags +bitexact -i $(TARGET_PATH)/tests/data/enhanced_flv_hevc.flv
> +
>  FATE_FFMPEG_FFPROBE += $(FATE_FLVENC_FFMPEG_FFPROBE-yes)
>  fate-flvenc: $(FATE_FLVENC_FFMPEG_FFPROBE-yes)
> diff --git a/tests/ref/fate/enhanced-flv-hevc b/tests/ref/fate/enhanced-flv-hevc
> new file mode 100644
> index 0000000000..ee53ccf982
> --- /dev/null
> +++ b/tests/ref/fate/enhanced-flv-hevc
> @@ -0,0 +1,6 @@
> +#tb 0: 1/1
> +#media_type 0: video
> +#codec_id 0: rawvideo
> +#dimensions 0: 176x144
> +#sar 0: 1/1
> +0,          0,          0,        1,    38016, 0x01bce789

_______________________________________________
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".

  parent reply	other threads:[~2023-07-18 11:35 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   ` Andreas Rheinhardt [this message]
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
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=AS8P250MB074476AA5BBB4D1EEF1AEED88F38A@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM \
    --to=andreas.rheinhardt@outlook.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