commit 6f2a1a24fa319e5cec308ddbeddd1ca87e0e1b65
author Boris Galitsky <bgalitsky@apache.org> Wed Jan 18 20:36:42 2012 +0000
committer Boris Galitsky <bgalitsky@apache.org> Wed Jan 18 20:36:42 2012 +0000
tree b78d48287d1df46d711cfa98f305f7c03b35a3de
parent ff11418ed26cef30b816e0d65353aaa231ba0cb5

repaired appropriateness criterion for sentence inclusion into generated content

opennlp-similarity/src/main/java/opennlp/tools/textsimilarity/SentencePairMatchResult.java

diff --git a/opennlp-similarity/src/main/java/opennlp/tools/textsimilarity/SentencePairMatchResult.java b/opennlp-similarity/src/main/java/opennlp/tools/textsimilarity/SentencePairMatchResult.java
index 632e1ea..9c47602 100644
--- a/opennlp-similarity/src/main/java/opennlp/tools/textsimilarity/SentencePairMatchResult.java
+++ b/opennlp-similarity/src/main/java/opennlp/tools/textsimilarity/SentencePairMatchResult.java
@@ -18,11 +18,13 @@
 package opennlp.tools.textsimilarity;

 

 import java.util.List;

+import java.util.logging.Logger;

 

 import org.apache.commons.lang.StringUtils;

 

 public class SentencePairMatchResult {

   public List<List<ParseTreeChunk>> matchResult;

+  private static Logger LOG = Logger.getLogger("opennlp.tools.textsimilarity.SentencePairMatchResult");

 

   public List<List<ParseTreeChunk>> getMatchResult() {

     return matchResult;

@@ -67,7 +69,7 @@
     super();

     verbExists = false;

     imperativeVerb = false;

-    System.out.println("Assessing sentence for inclusion " + resForMinedSent1);

+    //LOG.info("Assessing sentence for inclusion " + resForMinedSent1);

     this.matchResult = matchResult;

     this.resForMinedSent1 = resForMinedSent1;

     for (LemmaPair word : resForMinedSent1) {

@@ -75,18 +77,19 @@
           && StringUtils.isAlpha(word.getLemma())) {// ||

                                                     // word.getPOS().startsWith("VP"))

         verbExists = true;

-        System.out.println("Found verb=" + word);

+        //LOG.info("Found verb=" + word);

       }

     }

     // various form of sales pitch: 'get something', or 'we offer'

-    if (resForMinedSent1.get(1).getLemma().startsWith("We")

-        || resForMinedSent1.get(2).getLemma().startsWith("We"))

+    if (resForMinedSent1.size()>2 && (resForMinedSent1.get(1).getLemma().startsWith("We")

+        || resForMinedSent1.get(2).getLemma().startsWith("We")))

       imperativeVerb = true;

+    

     for (LemmaPair word : resForMinedSent1) {

       if (word.getPOS().startsWith("VB") && word.getStartPos() < 1

           && word.getEndPos() < 1) {

         imperativeVerb = true;

-        System.out.println("Found imperative verb=" + word);

+        //LOG.info("Found imperative verb=" + word);

       }

     }