[mlpack-svn] r16350 - mlpack/tags/mlpack-1.0.8/src/mlpack/methods/nmf

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

Author: rcurtin
Date: Sun Mar  9 00:10:27 2014
New Revision: 16350

Log:
Ack, don't apply the change against the tags!


Modified:
   mlpack/tags/mlpack-1.0.8/src/mlpack/methods/nmf/mult_div_update_rules.hpp

Modified: mlpack/tags/mlpack-1.0.8/src/mlpack/methods/nmf/mult_div_update_rules.hpp
==============================================================================
--- mlpack/tags/mlpack-1.0.8/src/mlpack/methods/nmf/mult_div_update_rules.hpp	(original)
+++ mlpack/tags/mlpack-1.0.8/src/mlpack/methods/nmf/mult_div_update_rules.hpp	Sun Mar  9 00:10:27 2014
@@ -75,18 +75,10 @@
         t2.set_size(H.n_cols);
         for (size_t k = 0; k < t2.n_elem; ++k)
         {
-          // This may produce NaNs if V(i, k) = 0.
-          // Technically the math in the paper does not define what to do in
-          // this case, but considering the basic intent of the update rules,
-          // we'll make this modification and take t2(k) = 0.0.
           t2(k) = H(j, k) * V(i, k) / t1(i, k);
-          if (t2(k) != t2(k))
-            t2(k) = 0.0;
         }
 
-        // Only update if the sum is not going to be 0, so as to prevent a
-        // divide by zero.  If sum(H.row(j)) is 0, then t2 should be 0 too.
-        W(i, j) *= sum(t2) / sum(H.row(j));
+        W(i, j) = W(i, j) * sum(t2) / sum(H.row(j));
       }
     }
   }
@@ -134,18 +126,10 @@
         t2.set_size(W.n_rows);
         for (size_t k = 0; k < t2.n_elem; ++k)
         {
-          // This may produce NaNs if V(i, k) = 0.
-          // Technically the math in the paper does not define what to do in
-          // this case, but considering the basic intent of the update rules,
-          // we'll make this modification and take t2(k) = 0.0.
           t2(k) = W(k, i) * V(k, j) / t1(k, j);
-          if (t2(k) != t2(k))
-            t2(k) = 0.0;
         }
 
-        // Only update if the sum is not going to be 0, so as to prevent a
-        // divide by zero.  If sum(W.col(j)) is 0, then t2 should be 0 too.
-        H(i, j) *= sum(t2) / sum(W.col(i));
+        H(i,j) = H(i,j) * sum(t2) / sum(W.col(i));
       }
     }
   }