You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by jo...@apache.org on 2011/09/27 20:58:14 UTC

svn commit: r1176525 - in /incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder: MultiModelNameFinder.java NameFinderJob.java

Author: joern
Date: Tue Sep 27 18:58:13 2011
New Revision: 1176525

URL: http://svn.apache.org/viewvc?rev=1176525&view=rev
Log:
OPENNLP-235 Enabled recall boosting for already confirmed names again.

Modified:
    incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/MultiModelNameFinder.java
    incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/NameFinderJob.java

Modified: incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/MultiModelNameFinder.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/MultiModelNameFinder.java?rev=1176525&r1=1176524&r2=1176525&view=diff
==============================================================================
--- incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/MultiModelNameFinder.java (original)
+++ incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/MultiModelNameFinder.java Tue Sep 27 18:58:13 2011
@@ -78,10 +78,10 @@ public class MultiModelNameFinder implem
       
       // if token part of name only token, then 
       // its either start, or cont
-//      if (valid && nameOnlyTokens.contains(inputSequence[i])) {
-//          return outcome.endsWith(NameFinderME.START) || 
-//                  outcome.endsWith(NameFinderME.CONTINUE); 
-//      }
+      if (valid && nameOnlyTokens.contains(modelType + "-" + inputSequence[i])) {
+          return outcome.endsWith(NameFinderME.START) || 
+                  outcome.endsWith(NameFinderME.CONTINUE); 
+      }
       
       return valid;
     }
@@ -134,10 +134,9 @@ public class MultiModelNameFinder implem
 
   void setNameOnlyTokens(Set<String> nameOnlyTokens) {
     
-    // How to do this for different types ?!
-//    for (RestrictedSequencesValidator sequenceValidator :  sequenceValidators) {
-//      sequenceValidator.setNameOnlyTokens(nameOnlyTokens);
-//    }
+    for (RestrictedSequencesValidator sequenceValidator :  sequenceValidators) {
+      sequenceValidator.setNameOnlyTokens(nameOnlyTokens);
+    }
   }
   
   @Override

Modified: incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/NameFinderJob.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/NameFinderJob.java?rev=1176525&r1=1176524&r2=1176525&view=diff
==============================================================================
--- incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/NameFinderJob.java (original)
+++ incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/namefinder/NameFinderJob.java Tue Sep 27 18:58:13 2011
@@ -154,7 +154,7 @@ public class NameFinderJob extends Job {
               StringPattern pattern = StringPattern.recognize(tokenStrings[i]);
               
               if (pattern.isAllLetter() && tokenStrings[i].length() > 1) {
-            	  nameTokens.add(tokenStrings[i]);
+            	  nameTokens.add(verifiedName.getType() + "-" + tokenStrings[i]);
               }
             }
           }