[mlpack-svn] r16036 - mlpack/trunk/src/mlpack/methods/logistic_regression

[fastlab-svn at coffeetalk-1.cc.gatech.edu]

Author: rcurtin
Date: Thu Nov 14 13:12:57 2013
New Revision: 16036

Log:
No need to multiply by responses[i] or (1 - responses[i]) because those
quantities will always be 1.


Modified:
   mlpack/trunk/src/mlpack/methods/logistic_regression/logistic_regression_function.cpp

Modified: mlpack/trunk/src/mlpack/methods/logistic_regression/logistic_regression_function.cpp
==============================================================================
--- mlpack/trunk/src/mlpack/methods/logistic_regression/logistic_regression_function.cpp	(original)
+++ mlpack/trunk/src/mlpack/methods/logistic_regression/logistic_regression_function.cpp	Thu Nov 14 13:12:57 2013
@@ -68,9 +68,9 @@
   for (size_t i = 0; i < responses.n_elem; ++i)
   {
     if (responses[i] == 1)
-      result += responses[i] * log(sigmoid[i]);
+      result += log(sigmoid[i]);
     else
-      result += (1 - responses[i]) * log(1.0 - sigmoid[i]);
+      result += log(1.0 - sigmoid[i]);
   }
 
   // Invert the result, because it's a minimization.