[mlpack-git] master: Removes some redundant code, fixes comments (6d11521)

gitdub at mlpack.org gitdub at mlpack.org
Thu Jun 30 15:12:03 EDT 2016


Repository : https://github.com/mlpack/mlpack
On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/eaa7182ebed8cce3fd6191dc1f8170546ea297da...812048c7c6bee0b6c8d936677f23bbb5930c6cfc

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

commit 6d11521ae7854115625d2088cf9f3cc57f79e252
Author: Yannis Mentekidis <mentekid at gmail.com>
Date:   Tue Jun 14 13:20:14 2016 +0300

    Removes some redundant code, fixes comments


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

6d11521ae7854115625d2088cf9f3cc57f79e252
 src/mlpack/tests/lsh_test.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/mlpack/tests/lsh_test.cpp b/src/mlpack/tests/lsh_test.cpp
index c66c8ef..2d9f556 100644
--- a/src/mlpack/tests/lsh_test.cpp
+++ b/src/mlpack/tests/lsh_test.cpp
@@ -609,21 +609,16 @@ BOOST_AUTO_TEST_CASE(MultiprobeDeterministicTest)
 
   // get offset of each projection dimension. Since projection table is I(2),
   // offsets will map to offsets of actual dimensions. This is to enforce that
-  // q1 and q4 are right outside the bounding boxes of C1 and C4 respectively.
+  // q1 is right outside the bounding box of C1.
   arma::mat offsets = lshTest.Offsets();
 
 
   // two points near the clusters but outside their bins. q1's lowest scoring
-  // perturbation vector will map to C1 cluster's bin. q4's two-lowest scoring
-  // vectors will map to empty bins, and its third vector will map to C4.
+  // perturbation vectors will map to C1 cluster's bins.
   arma::mat q1;
   q1 << 1.1 << arma::endr << 3.3; // vector [1.1, 3.3]
   q1 += offsets;
 
-  arma::mat q4;
-  q4 << 3.3 << arma::endr << 1.1; // vector [3.3, 1.1]
-  q4 += offsets;
-
   arma::Mat<size_t> neighbors;
   arma::mat distances;
 




More information about the mlpack-git mailing list