shithub: libvpx

Info  •  Files  •  Log  •  Branches

Enable and fix resize_test for VP9

Added the resize_test unit test to the VP9 set.

Set g_in_frames = 0 to avoid a problem when the total
number of frames being encoded is smaller than
g_in_frames. In this case the test will not have
access to the encoded frames and will not be able to
compare them for testing for encoder/decoder mismatch.

Change-Id: I0d2ff8ef058de7002c5faa894ed6ea794d5f900b

--- a/test/resize_test.cc

+++ b/test/resize_test.cc

@@ -124,6 +124,13 @@

   ::libvpx_test::I420VideoSource video("hantro_collage_w352h288.yuv", 352, 288,

                                        30, 1, 0, 10);

   init_flags_ = VPX_CODEC_USE_PSNR;

+

+  // If the number of frames being encoded is smaller than g_lag_in_frames

+  // the encoded frame is unavailable using the current API. Comparing

+  // frames to detect mismatch would then not be possible. Set

+  // g_lag_in_frames = 0 to get around this.

+  cfg_.g_lag_in_frames = 0;

+

   // q picked such that initial keyframe on this clip is ~30dB PSNR

   cfg_.rc_min_quantizer = cfg_.rc_max_quantizer = 48;

   ASSERT_NO_FATAL_FAILURE(RunLoop(&video));

--- a/test/test.mk

+++ b/test/test.mk

@@ -24,7 +24,7 @@

 LIBVPX_TEST_SRCS-$(CONFIG_ENCODERS)    += i420_video_source.h

 LIBVPX_TEST_SRCS-$(CONFIG_VP8_ENCODER) += keyframe_test.cc

 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += borders_test.cc

-LIBVPX_TEST_SRCS-$(CONFIG_VP8_ENCODER) += resize_test.cc

+LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += resize_test.cc

 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += cpu_speed_test.cc

 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += vp9_lossless_test.cc