On 1/8/2025 8:49 AM, 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 <p...@frankplowman.com> --- Changes since v1: * Remove instantiation of empty $(SCALE_OPTS) in fate-vvc-frames-with-ltr. * Move fate-vvc-frames-with-ltr to collection of VVC test cases which require libswscale as it is a 10-bit sequence. --- tests/fate/vvc.mak | 8 +++++--- tests/ref/fate/vvc-frames-with-ltr | 25 +++++++++++++++++++++++++ 2 files changed, 30 insertions(+), 3 deletions(-) create mode 100644 tests/ref/fate/vvc-frames-with-ltr diff --git a/tests/fate/vvc.mak b/tests/fate/vvc.mak index fb176b7745..76a4255e27 100644 --- a/tests/fate/vvc.mak +++ b/tests/fate/vvc.mak @@ -43,11 +43,13 @@ $(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 may have been an empty string if used here, but "-pix_fmt yuv420p10le -vf scale" is still needed. Otherwise the test will fail on BE systems.
I amended the patch and pushed it. Thanks.
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
