You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by ma...@apache.org on 2013/10/23 01:04:59 UTC

svn commit: r1534841 - /opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java

Author: markg
Date: Tue Oct 22 23:04:59 2013
New Revision: 1534841

URL: http://svn.apache.org/r1534841
Log:
OPENNLP-608
GeoHashBinScorer. No apparent cause for compilation error, builds  locally as normal. Trying again.

Modified:
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java?rev=1534841&r1=1534840&r2=1534841&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/entitylinker/GeoHashBinScorer.java Tue Oct 22 23:04:59 2013
@@ -164,7 +164,7 @@ public class GeoHashBinScorer {
       Long n = list.get(i + 1);
       Long v = list.get(i);
       //Long percent = 100 - (v / n * 100);
-      long diff = n-v;
+      Long diff = n-v;
       if (diff > avg) {
         breaks.add(v);
       }
@@ -175,16 +175,16 @@ public class GeoHashBinScorer {
     Long lastBreak = -1L;
     for (Long br : breaks) {
       if (lastBreak == -1L) {
-        binToAmount.put(br, sets.subSet(0L, true, br, false));
+        binToAmount.put(br, sets.subSet(0L, br));
       } else {
-        binToAmount.put(br, sets.subSet(lastBreak, true, br, false));
+        binToAmount.put(br, sets.subSet(lastBreak, br));
       }
       lastBreak = br;
 
     }
     lastBreak = sets.higher(lastBreak);
     if (lastBreak != null) {
-      binToAmount.put(lastBreak, sets.subSet(lastBreak, true, sets.last(), true));
+      binToAmount.put(lastBreak, sets.subSet(lastBreak, sets.last()));
       if (binToAmount.get(lastBreak).isEmpty()) {
         binToAmount.get(lastBreak).add(lastBreak);
       }