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]);
}
}
}