Fix weighted sum to add the full image value

Author: michaelgruner

sobel_cpu.cc

@@ -56,7 +56,7 @@ sobel(const cv::Mat &input, cv::Mat &output) {
 
   // Compute MAG2 = X^2 + Y^2
   cv::Mat mag2;
-  cv::addWeighted(x2, 0.5, y2, 0.5, 0, mag2);
+  cv::addWeighted(x2, 1, y2, 1, 0, mag2);
 
   // Compute MAG = √ (X^2 + Y^2)
   cv::Mat mag;

sobel_gpu_1_naive.cc

@@ -69,7 +69,7 @@ sobel(const Filters &filters, const cv::Mat &input, cv::Mat &output) {
 
   // MAG2 = X^2 + Y^2
   cv::cuda::GpuMat gpu_mag2;
-  cv::cuda::addWeighted(gpu_x2, 0.5, gpu_y2, 0.5, 0, gpu_mag2);
+  cv::cuda::addWeighted(gpu_x2, 1, gpu_y2, 1, 0, gpu_mag2);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::GpuMat gpu_mag;

sobel_gpu_2_single_alloc.cc

@@ -77,7 +77,7 @@ sobel(const Filters &filters, GpuMemories &gpu,
   cv::cuda::pow(gpu.y, 2, gpu.y2);
 
   // MAG2 = X^2 + Y^2
-  cv::cuda::addWeighted(gpu.x2, 0.5, gpu.y2, 0.5, 0, gpu.mag2);
+  cv::cuda::addWeighted(gpu.x2, 1, gpu.y2, 1, 0, gpu.mag2);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::sqrt(gpu.mag2, gpu.mag);

sobel_gpu_3_pinned_mem.cc

@@ -77,7 +77,7 @@ sobel(const Filters &filters, GpuMemories &gpu,
   cv::cuda::pow(gpu.y, 2, gpu.y2);
 
   // MAG2 = X^2 + Y^2
-  cv::cuda::addWeighted(gpu.x2, 0.5, gpu.y2, 0.5, 0, gpu.mag2);
+  cv::cuda::addWeighted(gpu.x2, 1, gpu.y2, 1, 0, gpu.mag2);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::sqrt(gpu.mag2, gpu.mag);

sobel_gpu_4_shared_mem.cc

@@ -75,7 +75,7 @@ sobel(const Filters &filters, GpuMemories &gpu) {
   cv::cuda::pow(gpu.y, 2, gpu.y2);
 
   // MAG2 = X^2 + Y^2
-  cv::cuda::addWeighted(gpu.x2, 0.5, gpu.y2, 0.5, 0, gpu.mag2);
+  cv::cuda::addWeighted(gpu.x2, 1, gpu.y2, 1, 0, gpu.mag2);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::sqrt(gpu.mag2, gpu.mag);

sobel_gpu_5_pinned_mem_streams.cc

@@ -86,7 +86,7 @@ sobel(const Filters &filters, Streams &streams, GpuMemories &gpu,
   streams.y.waitForCompletion();
 
   // MAG2 = X^2 + Y^2
-  cv::cuda::addWeighted(gpu.x2, 0.5, gpu.y2, 0.5, 0, gpu.mag2, -1, streams.x);
+  cv::cuda::addWeighted(gpu.x2, 1, gpu.y2, 1, 0, gpu.mag2, -1, streams.x);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::sqrt(gpu.mag2, gpu.mag, streams.x);

sobel_gpu_5_shared_mem_streams.cc

@@ -84,7 +84,7 @@ sobel(const Filters &filters, Streams &streams, GpuMemories &gpu) {
   streams.y.waitForCompletion();
 
   // MAG2 = X^2 + Y^2
-  cv::cuda::addWeighted(gpu.x2, 0.5, gpu.y2, 0.5, 0, gpu.mag2, -1, streams.x);
+  cv::cuda::addWeighted(gpu.x2, 1, gpu.y2, 1, 0, gpu.mag2, -1, streams.x);
 
   // MAG = √(X^2 + Y^2)
   cv::cuda::sqrt(gpu.mag2, gpu.mag, streams.x);