gpu_hamming_distance fails unit test with fixed RNG; mark it NOT_IMPLEMENTED
authorJeff Donahue <jeff.donahue@gmail.com>
Tue, 8 Apr 2014 11:25:20 +0000 (04:25 -0700)
committerJeff Donahue <jeff.donahue@gmail.com>
Wed, 9 Apr 2014 03:17:17 +0000 (20:17 -0700)
with TODO to fix and disable its unit test

src/caffe/test/test_math_functions.cpp
src/caffe/util/math_functions.cu

index 2c2c062..3ddd320 100644 (file)
@@ -81,7 +81,8 @@ TYPED_TEST(MathFunctionsTest, TestHammingDistanceCPU) {
             caffe_cpu_hamming_distance<TypeParam>(n, x, y));
 }
 
-TYPED_TEST(MathFunctionsTest, TestHammingDistanceGPU) {
+// TODO: Fix caffe_gpu_hamming_distance and re-enable this test.
+TYPED_TEST(MathFunctionsTest, DISABLED_TestHammingDistanceGPU) {
   int n = this->blob_bottom_->count();
   const TypeParam* x = this->blob_bottom_->cpu_data();
   const TypeParam* y = this->blob_top_->cpu_data();
index 0ac10d9..40c4f50 100644 (file)
@@ -159,6 +159,9 @@ __global__ void popcll_kernel(const int n, const double* a,
 template <>
 uint32_t caffe_gpu_hamming_distance<float>(const int n, const float* x,
                                   const float* y) {
+  // TODO: Fix caffe_gpu_hamming_distance (see failing unit test
+  // TestHammingDistanceGPU in test_math_functions.cpp).
+  NOT_IMPLEMENTED;
   thrust::device_vector<uint8_t> popcounts(n);
   // NOLINT_NEXT_LINE(whitespace/operators)
   popc_kernel<<<CAFFE_GET_BLOCKS(n), CAFFE_CUDA_NUM_THREADS>>>(
@@ -170,6 +173,9 @@ uint32_t caffe_gpu_hamming_distance<float>(const int n, const float* x,
 template <>
 uint32_t caffe_gpu_hamming_distance<double>(const int n, const double* x,
                                    const double* y) {
+  // TODO: Fix caffe_gpu_hamming_distance (see failing unit test
+  // TestHammingDistanceGPU in test_math_functions.cpp).
+  NOT_IMPLEMENTED;
   thrust::device_vector<uint8_t> popcounts(n);
   // NOLINT_NEXT_LINE(whitespace/operators)
   popcll_kernel<<<CAFFE_GET_BLOCKS(n), CAFFE_CUDA_NUM_THREADS>>>(