[mlpack-git] mlpack-2.0.x: Added missing function argument (4776877)

gitdub at mlpack.org gitdub at mlpack.org
Wed Jul 20 13:30:03 EDT 2016


Repository : https://github.com/mlpack/mlpack
On branch  : mlpack-2.0.x
Link       : https://github.com/mlpack/mlpack/compare/e434bc4ac042534529a2a440a44d86935b4d7164...fc4195d27bb9e642356a384d1fa6fe10cbdf89a6

>---------------------------------------------------------------

commit 477687781ab34cf7082c4703bdf188047cc6cf22
Author: Peter Lehner <peter.lehner at dlr.de>
Date:   Wed May 4 15:59:59 2016 +0200

    Added missing function argument
    
    The probabilities variable is not passed to the fitter when running multiple trials.


>---------------------------------------------------------------

477687781ab34cf7082c4703bdf188047cc6cf22
 src/mlpack/methods/gmm/gmm_impl.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mlpack/methods/gmm/gmm_impl.hpp b/src/mlpack/methods/gmm/gmm_impl.hpp
index 2322b72..d71e205 100644
--- a/src/mlpack/methods/gmm/gmm_impl.hpp
+++ b/src/mlpack/methods/gmm/gmm_impl.hpp
@@ -161,7 +161,7 @@ double GMM::Train(const arma::mat& observations,
         weightsTrial = weightsOrig;
       }
 
-      fitter.Estimate(observations, distsTrial, weightsTrial, useExistingModel);
+      fitter.Estimate(observations, probabilities, distsTrial, weightsTrial, useExistingModel);
 
       // Check to see if the log-likelihood of this one is better.
       double newLikelihood = LogLikelihood(observations, distsTrial,




More information about the mlpack-git mailing list