Removed source of randomness in dump_data.c for --test mode, that might have been causing arch-specific differences; diff32.c return correct fail code

CMakeLists.txt

@@ -201,7 +201,7 @@ add_test(NAME feature_extraction
                         dump_data --test --c2pitch ${CMAKE_CURRENT_SOURCE_DIR}/wav/birch.wav birch.f32;
                         md5sum birch.f32;
                         md5sum birch_targ.f32;
-                        diff32 birch_targ.f32 birch.f32")
+                        diff32 --cont birch_targ.f32 birch.f32")
 add_test(NAME nnet2f32
          COMMAND sh -c "cd ${CMAKE_CURRENT_BINARY_DIR}; ./src/nnet2f32 t.f32")
 add_test(NAME SIMD_functions

src/diff32.c

@@ -7,6 +7,7 @@
 #include <getopt.h>
 
 #define NB_FEATURES 55
+#define THRESH 0.001
 
 int main(int argc, char *argv[]) {
     float fdiff, fdiff_tot=0.0;
@@ -64,13 +65,17 @@ int main(int argc, char *argv[]) {
             fdiff = fabs(f1[i]-f2[i]);
             fdiff_tot += fdiff;
             
-            if (isnan(fdiff) || (fdiff > 1E-3)) {
+            if (isnan(fdiff) || (fdiff > THRESH)) {
-                printf("f: %d i: %d %f %f %f\n", f, i, f1[i], f2[i], fdiff);
+                fprintf(stderr, "f: %d i: %d %f %f %f\n", f, i, f1[i], f2[i], fdiff);
-                if (cont == 0) exit(0);
+                if (cont == 0) exit(1);
             }
         }
         f++;
     }
     fprintf(stderr,"stride: %d f: %d fdiff_tot: %f\n", stride, f, fdiff_tot);
     fclose(file1); fclose(file2);
+    if (fdiff_tot < THRESH)
+        exit(0);
+    else
+        exit(1);
 }

src/dump_data.c

@@ -438,7 +438,9 @@ int main(int argc, char **argv) {
       g = f*speech_gain + (1-f)*old_speech_gain;
       x[i] *= g;
     }
-    for (i=0;i<FRAME_SIZE;i++) x[i] += rand()/(float)RAND_MAX - .5;
+    if (training) {
+        for (i=0;i<FRAME_SIZE;i++) x[i] += rand()/(float)RAND_MAX - .5;
+    }
     compute_frame_features(st, X, P, Ex, Ep, Exp, features, x, logmag);
 
     if (c2pitch_en) {