[mlpack-git] master: Refactor to use Center(). (d49e069)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Thu Jul 16 12:16:51 EDT 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/b4659b668021db631b3c8a48e3d735b513706fdc...015d79104286231ef70ea0ba1d99a6c397025b3e

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

commit d49e069374b12a8bec360d0f015b00f79d6ce80a
Author: Ryan Curtin <ryan at ratml.org>
Date:   Thu Jul 16 16:16:06 2015 +0000

    Refactor to use Center().


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

d49e069374b12a8bec360d0f015b00f79d6ce80a
 src/mlpack/tests/tree_test.cpp | 48 +++++++++++++++++++++---------------------
 1 file changed, 24 insertions(+), 24 deletions(-)

diff --git a/src/mlpack/tests/tree_test.cpp b/src/mlpack/tests/tree_test.cpp
index 8d4e6d3..e7af33c 100644
--- a/src/mlpack/tests/tree_test.cpp
+++ b/src/mlpack/tests/tree_test.cpp
@@ -121,9 +121,9 @@ BOOST_AUTO_TEST_CASE(HRectBoundClear)
 }
 
 /**
- * Ensure that we get the correct centroid for our bound.
+ * Ensure that we get the correct center for our bound.
  */
-BOOST_AUTO_TEST_CASE(HRectBoundCentroid)
+BOOST_AUTO_TEST_CASE(HRectBoundCenter)
 {
   // Create a simple 3-dimensional bound.
   HRectBound<2> b(3);
@@ -132,14 +132,14 @@ BOOST_AUTO_TEST_CASE(HRectBoundCentroid)
   b[1] = Range(-2.0, -1.0);
   b[2] = Range(-10.0, 50.0);
 
-  arma::vec centroid;
+  arma::vec center;
 
-  b.Centroid(centroid);
+  b.Center(center);
 
-  BOOST_REQUIRE_EQUAL(centroid.n_elem, 3);
-  BOOST_REQUIRE_CLOSE(centroid[0], 2.5, 1e-5);
-  BOOST_REQUIRE_CLOSE(centroid[1], -1.5, 1e-5);
-  BOOST_REQUIRE_CLOSE(centroid[2], 20.0, 1e-5);
+  BOOST_REQUIRE_EQUAL(center.n_elem, 3);
+  BOOST_REQUIRE_CLOSE(center[0], 2.5, 1e-5);
+  BOOST_REQUIRE_CLOSE(center[1], -1.5, 1e-5);
+  BOOST_REQUIRE_CLOSE(center[2], 20.0, 1e-5);
 }
 
 /**
@@ -1113,14 +1113,14 @@ BOOST_AUTO_TEST_CASE(FurthestPointDistanceTest)
       BOOST_REQUIRE_EQUAL(node->FurthestPointDistance(), 0.0);
     else
     {
-      // Get centroid.
-      arma::vec centroid;
-      node->Centroid(centroid);
+      // Get center.
+      arma::vec center;
+      node->Center(center);
 
       double maxDist = 0.0;
       for (size_t i = 0; i < node->NumPoints(); ++i)
       {
-        const double dist = metric::EuclideanDistance::Evaluate(centroid,
+        const double dist = metric::EuclideanDistance::Evaluate(center,
             dataset.col(node->Point(i)));
         if (dist > maxDist)
           maxDist = dist;
@@ -1164,13 +1164,13 @@ BOOST_AUTO_TEST_CASE(ParentDistanceTest)
     if (node->NumChildren() == 0)
       continue;
 
-    arma::vec centroid, leftCentroid, rightCentroid;
-    node->Centroid(centroid);
-    node->Left()->Centroid(leftCentroid);
-    node->Right()->Centroid(rightCentroid);
+    arma::vec center, leftCenter, rightCenter;
+    node->Center(center);
+    node->Left()->Center(leftCenter);
+    node->Right()->Center(rightCenter);
 
-    const double leftDistance = LMetric<2>::Evaluate(centroid, leftCentroid);
-    const double rightDistance = LMetric<2>::Evaluate(centroid, rightCentroid);
+    const double leftDistance = LMetric<2>::Evaluate(center, leftCenter);
+    const double rightDistance = LMetric<2>::Evaluate(center, rightCenter);
 
     BOOST_REQUIRE_CLOSE(leftDistance, node->Left()->ParentDistance(), 1e-5);
     BOOST_REQUIRE_CLOSE(rightDistance, node->Right()->ParentDistance(), 1e-5);
@@ -1207,13 +1207,13 @@ BOOST_AUTO_TEST_CASE(ParentDistanceTestWithMapping)
     if (node->NumChildren() == 0)
       continue;
 
-    arma::vec centroid, leftCentroid, rightCentroid;
-    node->Centroid(centroid);
-    node->Left()->Centroid(leftCentroid);
-    node->Right()->Centroid(rightCentroid);
+    arma::vec center, leftCenter, rightCenter;
+    node->Center(center);
+    node->Left()->Center(leftCenter);
+    node->Right()->Center(rightCenter);
 
-    const double leftDistance = LMetric<2>::Evaluate(centroid, leftCentroid);
-    const double rightDistance = LMetric<2>::Evaluate(centroid, rightCentroid);
+    const double leftDistance = LMetric<2>::Evaluate(center, leftCenter);
+    const double rightDistance = LMetric<2>::Evaluate(center, rightCenter);
 
     BOOST_REQUIRE_CLOSE(leftDistance, node->Left()->ParentDistance(), 1e-5);
     BOOST_REQUIRE_CLOSE(rightDistance, node->Right()->ParentDistance(), 1e-5);



More information about the mlpack-git mailing list