Author: dfilimon
Date: Fri May 17 13:44:32 2013
New Revision: 1483799

URL: http://svn.apache.org/r1483799
Log:
MAHOUT-1219: LSHSearcher not always faster than BruteSearcher

The existing "benchmark" is not conclusive enough as it doesn't take into
account the adaptive nature of the JVM, doesn't run the test multiple times and
should be replaced with something more thorough.
Removing LSHSearcher from the tests for now.


Modified:
    mahout/trunk/CHANGELOG
    
mahout/trunk/core/src/test/java/org/apache/mahout/math/neighborhood/SearchQualityTest.java

Modified: mahout/trunk/CHANGELOG
URL: 
http://svn.apache.org/viewvc/mahout/trunk/CHANGELOG?rev=1483799&r1=1483798&r2=1483799&view=diff
==============================================================================
--- mahout/trunk/CHANGELOG (original)
+++ mahout/trunk/CHANGELOG Fri May 17 13:44:32 2013
@@ -2,6 +2,8 @@ Mahout Change Log
 
 Release 0.8 - unreleased
 
+__MAHOUT-1219: Remove LSHSearcher from SearchQualityTest. It causes it to 
fail, but the failure is not very meaningful (dfilimon)
+
 __MAHOUT-1217: Nearest neighbor searchers sometimes fail to remove points: fix 
in FastProjectionSearch's searchFirst (dfilimon)
 
 __MAHOUT-1216: Add locality sensitive hashing and a LocalitySensitiveHash 
searcher (dfilimon)

Modified: 
mahout/trunk/core/src/test/java/org/apache/mahout/math/neighborhood/SearchQualityTest.java
URL: 
http://svn.apache.org/viewvc/mahout/trunk/core/src/test/java/org/apache/mahout/math/neighborhood/SearchQualityTest.java?rev=1483799&r1=1483798&r2=1483799&view=diff
==============================================================================
--- 
mahout/trunk/core/src/test/java/org/apache/mahout/math/neighborhood/SearchQualityTest.java
 (original)
+++ 
mahout/trunk/core/src/test/java/org/apache/mahout/math/neighborhood/SearchQualityTest.java
 Fri May 17 13:44:32 2013
@@ -60,9 +60,6 @@ public class SearchQualityTest {
         // SEARCH_SIZE = 5
         {new ProjectionSearch(distanceMeasure, 5, 5), dataPoints, queries, 
reference, referenceSearchFirst},
         {new FastProjectionSearch(distanceMeasure, 5, 5), dataPoints, queries, 
reference, referenceSearchFirst},
-        {new LocalitySensitiveHashSearch(distanceMeasure, 5), dataPoints, 
queries, reference, referenceSearchFirst},
-        // SEARCH_SIZE = 2
-        {new LocalitySensitiveHashSearch(distanceMeasure, 2), dataPoints, 
queries, reference, referenceSearchFirst},
     }
     );
   }


Reply via email to