You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@stanbol.apache.org by rw...@apache.org on 2013/04/24 12:54:26 UTC

svn commit: r1471367 - in /stanbol/trunk/enhancement-engines: sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/ sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/

Author: rwesten
Date: Wed Apr 24 10:54:25 2013
New Revision: 1471367

URL: http://svn.apache.org/r1471367
Log:
fix for STANBOL-1051 also for the Sentiment engines

Modified:
    stanbol/trunk/enhancement-engines/sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/Sentiment.java
    stanbol/trunk/enhancement-engines/sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/SentimentEngine.java

Modified: stanbol/trunk/enhancement-engines/sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/Sentiment.java
URL: http://svn.apache.org/viewvc/stanbol/trunk/enhancement-engines/sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/Sentiment.java?rev=1471367&r1=1471366&r2=1471367&view=diff
==============================================================================
--- stanbol/trunk/enhancement-engines/sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/Sentiment.java (original)
+++ stanbol/trunk/enhancement-engines/sentiment-summarization/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/summarize/Sentiment.java Wed Apr 24 10:54:25 2013
@@ -64,7 +64,8 @@ public class Sentiment {
         this.end = token.getEnd();
         List<Value<PosTag>> tags = token.getAnnotations(NlpAnnotations.POS_ANNOTATION);
         for(Value<PosTag> tag : tags){
-            if(tag.probability() >= MIN_POS_CONF || 
+            if(tag.probability() == Value.UNKNOWN_PROBABILITY ||
+                    tag.probability() >= MIN_POS_CONF || 
                     !Collections.disjoint(tag.value().getCategories(),PREF_LEX_CAT)){
                 posTag = tag.value();
                 break;

Modified: stanbol/trunk/enhancement-engines/sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/SentimentEngine.java
URL: http://svn.apache.org/viewvc/stanbol/trunk/enhancement-engines/sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/SentimentEngine.java?rev=1471367&r1=1471366&r2=1471367&view=diff
==============================================================================
--- stanbol/trunk/enhancement-engines/sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/SentimentEngine.java (original)
+++ stanbol/trunk/enhancement-engines/sentiment-wordclassifier/src/main/java/org/apache/stanbol/enhancer/engines/sentiment/services/SentimentEngine.java Wed Apr 24 10:54:25 2013
@@ -272,8 +272,10 @@ public class SentimentEngine  extends Ab
                     Value<PosTag> value = posTags.next();
                     PosTag tag = value.value();
                     boolean state = classifier.isAdjective(tag) || classifier.isNoun(tag);
-                    ignore = !state && value.probability() >= minPOSConfidence;
-                    process = state && value.probability() >= (minPOSConfidence/2.0);
+                    ignore = !state && (value.probability() == Value.UNKNOWN_PROBABILITY ||
+                            value.probability() >= minPOSConfidence);
+                    process = state && (value.probability() == Value.UNKNOWN_PROBABILITY ||
+                            value.probability() >= (minPOSConfidence/2.0));
                 }
             } //else process all tokens ... no POS tag checking needed
             if(process){