[mlpack-svn] r14895 - mlpack/trunk/src/mlpack/methods/gmm

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

Author: rcurtin
Date: 2013-04-12 13:58:08 -0400 (Fri, 12 Apr 2013)
New Revision: 14895

Modified:
   mlpack/trunk/src/mlpack/methods/gmm/em_fit_impl.hpp
Log:
I think this warning can be somewhat misleading, so it should be removed.


Modified: mlpack/trunk/src/mlpack/methods/gmm/em_fit_impl.hpp
===================================================================
--- mlpack/trunk/src/mlpack/methods/gmm/em_fit_impl.hpp	2013-04-12 17:23:16 UTC (rev 14894)
+++ mlpack/trunk/src/mlpack/methods/gmm/em_fit_impl.hpp	2013-04-12 17:58:08 UTC (rev 14895)
@@ -77,7 +77,6 @@
 
     // Calculate the new value of the means using the updated conditional
     // probabilities.
-    size_t nonPositive = 0; // Count the number of non-positive covariances.
     for (size_t i = 0; i < means.size(); i++)
     {
       // Don't update if there's no probability of the Gaussian having points.
@@ -111,10 +110,6 @@
       }
     }
 
-    if (nonPositive > 0)
-      Log::Warn << nonPositive << " covariance matrices are very small. "
-          << "Consider reducing the number of Gaussians." << std::endl;
-
     // Calculate the new values for omega using the updated conditional
     // probabilities.
     weights = probRowSums / observations.n_cols;