You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ctakes.apache.org by se...@apache.org on 2017/08/20 22:46:16 UTC

svn commit: r1805586 - /ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java

Author: seanfinan
Date: Sun Aug 20 22:46:16 2017
New Revision: 1805586

URL: http://svn.apache.org/viewvc?rev=1805586&view=rev
Log:
Preliminary check for blacklist

Modified:
    ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java

Modified: ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java
URL: http://svn.apache.org/viewvc/ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java?rev=1805586&r1=1805585&r2=1805586&view=diff
==============================================================================
--- ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java (original)
+++ ctakes/trunk/ctakes-dictionary-lookup-fast/src/main/java/org/apache/ctakes/dictionary/lookup2/consumer/DefaultTermConsumer.java Sun Aug 20 22:46:16 2017
@@ -62,7 +62,7 @@ final public class DefaultTermConsumer e
    private final CollectionMap<Integer, String, Set<String>> _blacklists = new HashSetMap<>();
    private final CollectionMap<Integer, String, Set<String>> _csBlacklists = new HashSetMap<>();
 
-
+   private boolean _checkBlacklist;
 
    public DefaultTermConsumer( final UimaContext uimaContext, final Properties properties ) {
       this( uimaContext, properties, new DefaultUmlsConceptCreator() );
@@ -86,6 +86,7 @@ final public class DefaultTermConsumer e
       if ( csBlacklistPath != null && !csBlacklistPath.equals( EnvironmentVariable.NOT_PRESENT ) ) {
          loadBlacklist( csBlacklistPath, _csBlacklists, true );
       }
+      _checkBlacklist = !_blacklists.isEmpty() || !_csBlacklists.isEmpty();
    }
 
    /**
@@ -145,6 +146,9 @@ final public class DefaultTermConsumer e
     * @return true if the candidate text is in the blacklist for the semantic type
     */
    private boolean inBlacklist( final int cTakesSemantic, final JCas jCas, final TextSpan textSpan ) {
+      if ( !_checkBlacklist ) {
+         return false;
+      }
       final String text = jCas.getDocumentText().substring( textSpan.getStart(), textSpan.getEnd() ).trim();
       return _csBlacklists.containsValue( cTakesSemantic, text )
             || _blacklists.containsValue( cTakesSemantic, text.toLowerCase() );