From 3dbc78b1342e3053b7f8a62fc5839c927e91699f Mon Sep 17 00:00:00 2001
From: Yaowu Xu <yaowu@google.com>
Date: Tue, 26 Feb 2013 09:07:49 -0800
Subject: [PATCH] Enable 32x32 dct tests

Also
1. Removed the test code for fDCT from the iDCT test.
2. changed the criteria of round trip error to be below 1/block, this
is quite strict comparing to smaller transforms when size differences
are accounted for.

Change-Id: Idb46a6380b04c93fc8e2845c75f5a850366b0090
---
 test/dct32x32_test.cc | 13 ++-----------
 test/test.mk          |  2 +-
 2 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/test/dct32x32_test.cc b/test/dct32x32_test.cc
index d14e0aa2f0..a565270993 100644
--- a/test/dct32x32_test.cc
+++ b/test/dct32x32_test.cc
@@ -115,15 +115,6 @@ TEST(VP9Idct32x32Test, AccuracyCheck) {
           << "Error: 3x32 IDCT has error " << error
           << " at index " << j;
     }
-
-    vp9_short_fdct32x32_c(in, out_c, 64);
-    for (int j = 0; j < 1024; ++j) {
-      const double diff = coeff[j] - out_c[j];
-      const double error = diff * diff;
-      EXPECT_GE(1.0, error)
-          << "Error: 32x32 FDCT has error " << error
-          << " at index " << j;
-    }
   }
 }
 
@@ -157,8 +148,8 @@ TEST(VP9Fdct32x32Test, AccuracyCheck) {
   EXPECT_GE(1u, max_error)
       << "Error: 32x32 FDCT/IDCT has an individual roundtrip error > 1";
 
-  EXPECT_GE(count_test_block/10, total_error)
-      << "Error: 32x32 FDCT/IDCT has average roundtrip error > 1/10 per block";
+  EXPECT_GE(count_test_block, total_error)
+      << "Error: 32x32 FDCT/IDCT has average roundtrip error > 1 per block";
 }
 
 TEST(VP9Fdct32x32Test, CoeffSizeCheck) {
diff --git a/test/test.mk b/test/test.mk
index d879f9fb28..3d56bd838c 100644
--- a/test/test.mk
+++ b/test/test.mk
@@ -76,7 +76,7 @@ LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += fdct4x4_test.cc
 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += fdct8x8_test.cc
 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += dct16x16_test.cc
 LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += variance_test.cc
-#LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += dct32x32_test.cc
+LIBVPX_TEST_SRCS-$(CONFIG_VP9_ENCODER) += dct32x32_test.cc
 
 endif # VP9
 
-- 
GitLab