From a99050f0c8cb0315fa31e3d1fa3e38594fe5e40a Mon Sep 17 00:00:00 2001
From: Joseph Redmon <pjreddie@gmail.com>
Date: Mon, 08 Dec 2014 04:16:21 +0000
Subject: [PATCH] Some fixes to momentum

---
 src/convolutional_layer.c |  122 +++++++++++++++-------------------------
 1 files changed, 46 insertions(+), 76 deletions(-)

diff --git a/src/convolutional_layer.c b/src/convolutional_layer.c
index 42f4f21..5b4e0b5 100644
--- a/src/convolutional_layer.c
+++ b/src/convolutional_layer.c
@@ -59,34 +59,31 @@
 
     layer->filters = calloc(c*n*size*size, sizeof(float));
     layer->filter_updates = calloc(c*n*size*size, sizeof(float));
-    layer->filter_momentum = calloc(c*n*size*size, sizeof(float));
 
     layer->biases = calloc(n, sizeof(float));
     layer->bias_updates = calloc(n, sizeof(float));
-    layer->bias_momentum = calloc(n, sizeof(float));
     float scale = 1./(size*size*c);
-    scale = .05;
-    for(i = 0; i < c*n*size*size; ++i) layer->filters[i] = scale*2*(rand_uniform()-.5);
+    scale = .01;
+    for(i = 0; i < c*n*size*size; ++i) layer->filters[i] = scale*rand_normal();
     for(i = 0; i < n; ++i){
         //layer->biases[i] = rand_normal()*scale + scale;
-        layer->biases[i] = .5;
+        layer->biases[i] = .01;
     }
     int out_h = convolutional_out_height(*layer);
     int out_w = convolutional_out_width(*layer);
 
-    layer->col_image = calloc(layer->batch*out_h*out_w*size*size*c, sizeof(float));
+    layer->col_image = calloc(out_h*out_w*size*size*c, sizeof(float));
     layer->output = calloc(layer->batch*out_h * out_w * n, sizeof(float));
     layer->delta  = calloc(layer->batch*out_h * out_w * n, sizeof(float));
+
     #ifdef GPU
     layer->filters_cl = cl_make_array(layer->filters, c*n*size*size);
     layer->filter_updates_cl = cl_make_array(layer->filter_updates, c*n*size*size);
-    layer->filter_momentum_cl = cl_make_array(layer->filter_momentum, c*n*size*size);
 
     layer->biases_cl = cl_make_array(layer->biases, n);
     layer->bias_updates_cl = cl_make_array(layer->bias_updates, n);
-    layer->bias_momentum_cl = cl_make_array(layer->bias_momentum, n);
 
-    layer->col_image_cl = cl_make_array(layer->col_image, layer->batch*out_h*out_w*size*size*c);
+    layer->col_image_cl = cl_make_array(layer->col_image, out_h*out_w*size*size*c);
     layer->delta_cl = cl_make_array(layer->delta, layer->batch*out_h*out_w*n);
     layer->output_cl = cl_make_array(layer->output, layer->batch*out_h*out_w*n);
     #endif
@@ -106,7 +103,7 @@
     int out_w = convolutional_out_width(*layer);
 
     layer->col_image = realloc(layer->col_image,
-                                layer->batch*out_h*out_w*layer->size*layer->size*layer->c*sizeof(float));
+                                out_h*out_w*layer->size*layer->size*layer->c*sizeof(float));
     layer->output = realloc(layer->output,
                                 layer->batch*out_h * out_w * layer->n*sizeof(float));
     layer->delta  = realloc(layer->delta,
@@ -143,13 +140,13 @@
     float *b = layer.col_image;
     float *c = layer.output;
 
-    im2col_cpu(in, layer.batch, layer.c, layer.h, layer.w, 
-        layer.size, layer.stride, layer.pad, b);
 
     for(i = 0; i < layer.batch; ++i){
+        im2col_cpu(in, layer.c, layer.h, layer.w, 
+            layer.size, layer.stride, layer.pad, b);
         gemm(0,0,m,n,k,1,a,k,b,n,1,c,n);
-        b += k*n;
         c += n*m;
+        in += layer.c*layer.h*layer.w;
     }
     activate_array(layer.output, m*n*layer.batch, layer.activation);
 }
@@ -166,7 +163,7 @@
     }
 }
 
-void backward_convolutional_layer(convolutional_layer layer, float *delta)
+void backward_convolutional_layer(convolutional_layer layer, float *in, float *delta)
 {
     int i;
     int m = layer.n;
@@ -176,35 +173,28 @@
     gradient_array(layer.output, m*k*layer.batch, layer.activation, layer.delta);
     learn_bias_convolutional_layer(layer);
 
-    float *a = layer.delta;
-    float *b = layer.col_image;
-    float *c = layer.filter_updates;
+    if(delta) memset(delta, 0, layer.batch*layer.h*layer.w*layer.c*sizeof(float));
 
     for(i = 0; i < layer.batch; ++i){
+        float *a = layer.delta + i*m*k;
+        float *b = layer.col_image;
+        float *c = layer.filter_updates;
+
+        float *im = in+i*layer.c*layer.h*layer.w;
+
+        im2col_cpu(im, layer.c, layer.h, layer.w, 
+                layer.size, layer.stride, layer.pad, b);
         gemm(0,1,m,n,k,1,a,k,b,k,1,c,n);
-        a += m*k;
-        b += k*n;
-    }
 
-    if(delta){
-        m = layer.size*layer.size*layer.c;
-        k = layer.n;
-        n = convolutional_out_height(layer)*
-            convolutional_out_width(layer);
+        if(delta){
+            a = layer.filters;
+            b = layer.delta + i*m*k;
+            c = layer.col_image;
 
-        a = layer.filters;
-        b = layer.delta;
-        c = layer.col_image;
+            gemm(1,0,n,k,m,1,a,n,b,k,0,c,k);
 
-        for(i = 0; i < layer.batch; ++i){
-            gemm(1,0,m,n,k,1,a,m,b,n,0,c,n);
-            b += k*n;
-            c += m*n;
+            col2im_cpu(layer.col_image, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, delta+i*layer.c*layer.h*layer.w);
         }
-
-        memset(delta, 0, layer.batch*layer.h*layer.w*layer.c*sizeof(float));
-
-        col2im_cpu(layer.col_image, layer.batch, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, delta);
     }
 }
 
@@ -214,7 +204,7 @@
     axpy_cpu(layer.n, layer.learning_rate, layer.bias_updates, 1, layer.biases, 1);
     scal_cpu(layer.n, layer.momentum, layer.bias_updates, 1);
 
-    scal_cpu(size, 1.-layer.learning_rate*layer.decay, layer.filters, 1);
+    axpy_cpu(size, -layer.decay, layer.filters, 1, layer.filter_updates, 1);
     axpy_cpu(size, layer.learning_rate, layer.filter_updates, 1, layer.filters, 1);
     scal_cpu(size, layer.momentum, layer.filter_updates, 1);
 }
@@ -304,7 +294,7 @@
 
     const size_t global_size[] = {layer.n};
 
-    clEnqueueNDRangeKernel(queue, kernel, 1, 0, global_size, 0, 0, 0, 0);
+    cl.error = clEnqueueNDRangeKernel(queue, kernel, 1, 0, global_size, 0, 0, 0, 0);
     check_error(cl);
 }
 
@@ -336,9 +326,9 @@
     cl.error = clSetKernelArg(kernel, i++, sizeof(layer.output_cl), (void*) &layer.output_cl);
     check_error(cl);
 
-    const size_t global_size[] = {layer.batch, layer.n*size};
+    const size_t global_size[] = {layer.n*size, layer.batch};
 
-    clEnqueueNDRangeKernel(queue, kernel, 2, 0, global_size, 0, 0, 0, 0);
+    cl.error = clEnqueueNDRangeKernel(queue, kernel, 2, 0, global_size, 0, 0, 0, 0);
     check_error(cl);
 }
 
@@ -354,36 +344,17 @@
 
     bias_output_gpu(layer);
 
-    #ifdef TIMEIT
-    clock_t time = clock();
-    printf("Forward\n");
-    #endif
-
-    im2col_ongpu(in, layer.batch, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, layer.col_image_cl);
-
-    #ifdef TIMEIT
-    clFinish(cl.queue);
-    printf("Im2col %f\n", sec(clock()-time));
-    time = clock();
-    #endif
-
     for(i = 0; i < layer.batch; ++i){
+        im2col_ongpu(in, i*layer.c*layer.h*layer.w, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, layer.col_image_cl);
         cl_mem a = layer.filters_cl;
         cl_mem b = layer.col_image_cl;
         cl_mem c = layer.output_cl;
-        gemm_ongpu_offset(0,0,m,n,k,1.,a,0,k,b,i*k*n,n,1.,c,i*m*n,n);
+        gemm_ongpu_offset(0,0,m,n,k,1.,a,0,k,b,0,n,1.,c,i*m*n,n);
     }
-    #ifdef TIMEIT
-    clFinish(cl.queue);
-    printf("Gemm %f\n", sec(clock()-time));
-    #endif
     activate_array_ongpu(layer.output_cl, m*n*layer.batch, layer.activation);
-    #ifdef TIMEIT
-    cl_read_array(layer.output_cl, layer.output, m*n*layer.batch);
-    #endif
 }
 
-void backward_convolutional_layer_gpu(convolutional_layer layer, cl_mem delta_cl)
+void backward_convolutional_layer_gpu(convolutional_layer layer, cl_mem in, cl_mem delta_cl)
 {
     int i;
     int m = layer.n;
@@ -393,31 +364,26 @@
     gradient_array_ongpu(layer.output_cl, m*k*layer.batch, layer.activation, layer.delta_cl);
     learn_bias_convolutional_layer_ongpu(layer);
 
+    if(delta_cl) scal_ongpu(layer.batch*layer.h*layer.w*layer.c, 0, delta_cl, 1);
+
     for(i = 0; i < layer.batch; ++i){
         cl_mem a = layer.delta_cl;
         cl_mem b = layer.col_image_cl;
         cl_mem c = layer.filter_updates_cl;
 
-        gemm_ongpu_offset(0,1,m,n,k,1,a,i*m*k,k,b,i*k*n,k,1,c,0,n);
-    }
-    //cl_read_array(layer.delta_cl, layer.delta, m*k*layer.batch);
+        im2col_ongpu(in, i*layer.c*layer.h*layer.w, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, layer.col_image_cl);
+        gemm_ongpu_offset(0,1,m,n,k,1,a,i*m*k,k,b,0,k,1,c,0,n);
 
-    if(delta_cl){
-        m = layer.size*layer.size*layer.c;
-        k = layer.n;
-        n = convolutional_out_height(layer)*
-            convolutional_out_width(layer);
+        if(delta_cl){
 
-        for(i = 0; i < layer.batch; ++i){
             cl_mem a = layer.filters_cl;
             cl_mem b = layer.delta_cl;
             cl_mem c = layer.col_image_cl;
 
-            gemm_ongpu_offset(1,0,m,n,k,1,a,0,m,b,i*k*n,n,0,c,i*m*n,n);
-        }
+            gemm_ongpu_offset(1,0,n,k,m,1,a,0,n,b,i*k*m,k,0,c,0,k);
 
-        scal_ongpu(layer.batch*layer.h*layer.w*layer.c,0,delta_cl, 1);
-        col2im_ongpu(layer.col_image_cl, layer.batch, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, delta_cl);
+            col2im_ongpu(layer.col_image_cl, i*layer.c*layer.h*layer.w, layer.c,  layer.h,  layer.w,  layer.size,  layer.stride, layer.pad, delta_cl);
+        }
     }
 }
 
@@ -425,12 +391,16 @@
 {
     cl_read_array(layer.filters_cl, layer.filters, layer.c*layer.n*layer.size*layer.size);
     cl_read_array(layer.biases_cl, layer.biases, layer.n);
+    cl_read_array(layer.filter_updates_cl, layer.filter_updates, layer.c*layer.n*layer.size*layer.size);
+    cl_read_array(layer.bias_updates_cl, layer.bias_updates, layer.n);
 }
 
 void push_convolutional_layer(convolutional_layer layer)
 {
     cl_write_array(layer.filters_cl, layer.filters, layer.c*layer.n*layer.size*layer.size);
     cl_write_array(layer.biases_cl, layer.biases, layer.n);
+    cl_write_array(layer.filter_updates_cl, layer.filter_updates, layer.c*layer.n*layer.size*layer.size);
+    cl_write_array(layer.bias_updates_cl, layer.bias_updates, layer.n);
 }
 
 void update_convolutional_layer_gpu(convolutional_layer layer)
@@ -439,7 +409,7 @@
     axpy_ongpu(layer.n, layer.learning_rate, layer.bias_updates_cl, 1, layer.biases_cl, 1);
     scal_ongpu(layer.n,layer.momentum, layer.bias_updates_cl, 1);
 
-    scal_ongpu(size, 1.-layer.learning_rate*layer.decay, layer.filters_cl, 1);
+    axpy_ongpu(size, -layer.decay, layer.filters_cl, 1, layer.filter_updates_cl, 1);
     axpy_ongpu(size, layer.learning_rate, layer.filter_updates_cl, 1, layer.filters_cl, 1);
     scal_ongpu(size, layer.momentum, layer.filter_updates_cl, 1);
     pull_convolutional_layer(layer);

--
Gitblit v1.10.0