[mlpack-git] master: Style fixes (38fc485)

gitdub at mlpack.org gitdub at mlpack.org
Thu Jul 21 11:07:24 EDT 2016


Repository : https://github.com/mlpack/mlpack
On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/ba850f782a53c5a77b7985f7647f609bd96cb5e7...2c026d838925df436d967439899813da5d34c702

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

commit 38fc485c0599e60700ce1701a90a5356a80b9caf
Author: Yannis Mentekidis <mentekid at gmail.com>
Date:   Thu Jul 21 16:07:24 2016 +0100

    Style fixes


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

38fc485c0599e60700ce1701a90a5356a80b9caf
 src/mlpack/core/dists/gamma_distribution.cpp | 2 --
 src/mlpack/core/dists/gamma_distribution.hpp | 6 +++---
 src/mlpack/tests/distribution_test.cpp       | 7 ++++---
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/mlpack/core/dists/gamma_distribution.cpp b/src/mlpack/core/dists/gamma_distribution.cpp
index 5de0630..4d93917 100644
--- a/src/mlpack/core/dists/gamma_distribution.cpp
+++ b/src/mlpack/core/dists/gamma_distribution.cpp
@@ -5,14 +5,12 @@
  * Implementation of the methods of GammaDistribution.
  */
 #include "gamma_distribution.hpp"
-
 #include <boost/math/special_functions/digamma.hpp>
 //#include <boost/math/special_functions/trigamma.hpp> // Moved to prereqs.hpp
 
 using namespace mlpack;
 using namespace mlpack::distribution;
 
-
 // Returns true if computation converged.
 inline bool GammaDistribution::converged(double aOld, double aNew)
 {
diff --git a/src/mlpack/core/dists/gamma_distribution.hpp b/src/mlpack/core/dists/gamma_distribution.hpp
index f12086e..18859ef 100644
--- a/src/mlpack/core/dists/gamma_distribution.hpp
+++ b/src/mlpack/core/dists/gamma_distribution.hpp
@@ -43,12 +43,12 @@ class GammaDistribution
     ~GammaDistribution() {};
 
     /**
-     * This function trains (fits distribution parameters) to new data or data
-     * object was constructed with.
+     * This function trains (fits distribution parameters) to new data or the
+     * dataset the object owns.
      *
      * @param rdata Reference data to fit parameters to. If not specified,
      *    reference data will be used. Results are stored in the alpha and beta
-     *    vectors.
+     *    vectors (old results are removed).
      */
     void Train(const arma::mat& rdata = arma::mat());
 
diff --git a/src/mlpack/tests/distribution_test.cpp b/src/mlpack/tests/distribution_test.cpp
index e605134..3d754de 100644
--- a/src/mlpack/tests/distribution_test.cpp
+++ b/src/mlpack/tests/distribution_test.cpp
@@ -397,7 +397,8 @@ BOOST_AUTO_TEST_CASE(GammaDistributionConstructorTest)
 {
   // Empty constructor test. Make sure reference set is empty.
   GammaDistribution gDist;
-  BOOST_REQUIRE_EQUAL(arma::size(gDist.ReferenceSet()), arma::size(arma::mat()));
+  BOOST_REQUIRE_EQUAL(arma::size(gDist.ReferenceSet()),
+      arma::size(arma::mat()));
 
   // Parameterized constructor test. Make sure reference set is passed
   // correctly.
@@ -420,7 +421,8 @@ BOOST_AUTO_TEST_CASE(GammaDistributionConstructorTest)
 
   // Create Gamma object.
   GammaDistribution gDist2(rdata);
-  BOOST_REQUIRE_EQUAL(arma::size(gDist2.ReferenceSet()), arma::size(arma::mat(d, N)));
+  BOOST_REQUIRE_EQUAL(arma::size(gDist2.ReferenceSet()),
+      arma::size(arma::mat(d, N)));
 }
 
 /**
@@ -470,7 +472,6 @@ BOOST_AUTO_TEST_CASE(GammaDistributionTrainTest)
   // Training must estimate d' pairs of alpha and beta parameters.
   BOOST_REQUIRE_EQUAL(gDist.Alpha().n_elem, d2);
   BOOST_REQUIRE_EQUAL(gDist.Beta().n_elem, d2);
-
 }
 
 /**




More information about the mlpack-git mailing list