[FFmpeg-devel] [PATCH] fate/vvc: Add vvc-frames-with-ltr.vvc
James Almer
jamrial at gmail.com
Wed Jan 8 00:58:49 EET 2025
On 1/7/2025 7:19 PM, Frank Plowman wrote:
> This sample is rather difficult, containing a lot of subtle edge cases
> which revealed errors in the VVC decoder. It covers 88.4% of lines in
> libavcodec/vvc and brings the line coverage of the entire VVC fate suite
> from 96.3% to 97.2%.
>
> Signed-off-by: Frank Plowman <post at frankplowman.com>
> ---
> Sorry this sat on the FATE rsync for so long before adding this test.
> I hadn't spotted the race it uncovered before requesting its upload
> and that took a little while to debug.
> ---
> tests/fate/vvc.mak | 3 ++-
> tests/ref/fate/vvc-frames-with-ltr | 25 +++++++++++++++++++++++++
> 2 files changed, 27 insertions(+), 1 deletion(-)
> create mode 100644 tests/ref/fate/vvc-frames-with-ltr
>
> diff --git a/tests/fate/vvc.mak b/tests/fate/vvc.mak
> index fb176b7745..c37fe7d3d4 100644
> --- a/tests/fate/vvc.mak
> +++ b/tests/fate/vvc.mak
> @@ -43,8 +43,9 @@ $(VVC_TESTS_10BIT): SCALE_OPTS := -pix_fmt yuv420p10le -vf scale
> $(VVC_TESTS_444_10BIT): SCALE_OPTS := -pix_fmt yuv444p10le -vf scale
> fate-vvc-conformance-%: CMD = framecrc -c:v vvc -i $(TARGET_SAMPLES)/vvc-conformance/$(subst fate-vvc-conformance-,,$(@)).bit $(SCALE_OPTS)
> fate-vvc-output-ref: CMD = framecrc -c:v vvc -i $(TARGET_SAMPLES)/vvc/Hierarchical.bit $(SCALE_OPTS)
> +fate-vvc-frames-with-ltr: CMD = framecrc -c:v vvc -i $(TARGET_SAMPLES)/vvc/vvc_frames_with_ltr.vvc $(SCALE_OPTS)
$(SCALE_OPTS) here is an empty string. It's only set for the conformance
tests above.
That also applies to fate-vvc-output-ref, btw. It's also an empty string
for it, but it doesn't really matter as yuv420p is the implied output
format anyway.
>
> -FATE_VVC-$(call FRAMECRC, VVC, VVC, VVC_PARSER) += $(VVC_TESTS_8BIT) fate-vvc-output-ref
> +FATE_VVC-$(call FRAMECRC, VVC, VVC, VVC_PARSER) += $(VVC_TESTS_8BIT) fate-vvc-output-ref fate-vvc-frames-with-ltr
vvc_frames_with_ltr.vvc is a 10bit sample, so you need to add it to the
list below, to ensure it depends on libswscale.
> FATE_VVC-$(call FRAMECRC, VVC, VVC, VVC_PARSER SCALE_FILTER) += \
> $(VVC_TESTS_10BIT) \
> $(VVC_TESTS_444_10BIT) \
> diff --git a/tests/ref/fate/vvc-frames-with-ltr b/tests/ref/fate/vvc-frames-with-ltr
> new file mode 100644
> index 0000000000..357af8e732
> --- /dev/null
> +++ b/tests/ref/fate/vvc-frames-with-ltr
> @@ -0,0 +1,25 @@
> +#tb 0: 1/25
> +#media_type 0: video
> +#codec_id 0: rawvideo
> +#dimensions 0: 800x872
> +#sar 0: 0/1
> +0, 0, 0, 1, 2092800, 0xce6d9094
> +0, 1, 1, 1, 2092800, 0x599f9867
> +0, 2, 2, 1, 2092800, 0x6d068c57
> +0, 3, 3, 1, 2092800, 0x90e4d43c
> +0, 4, 4, 1, 2092800, 0x0e2c5d62
> +0, 5, 5, 1, 2092800, 0x4d6c0288
> +0, 6, 6, 1, 2092800, 0x6334cd3f
> +0, 7, 7, 1, 2092800, 0xd76d4407
> +0, 8, 8, 1, 2092800, 0x9bbc1e0b
> +0, 9, 9, 1, 2092800, 0xad3851b1
> +0, 10, 10, 1, 2092800, 0x37876792
> +0, 11, 11, 1, 2092800, 0x1f55a6b6
> +0, 12, 12, 1, 2092800, 0x94b37de2
> +0, 13, 13, 1, 2092800, 0x35e81d80
> +0, 14, 14, 1, 2092800, 0xa6c0de46
> +0, 15, 15, 1, 2092800, 0xb1a64e7a
> +0, 16, 16, 1, 2092800, 0xc02796a7
> +0, 17, 17, 1, 2092800, 0x30ab91c4
> +0, 18, 18, 1, 2092800, 0xe480a19f
> +0, 19, 19, 1, 2092800, 0x2818666d
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250107/f90b99d5/attachment.sig>
More information about the ffmpeg-devel
mailing list