This is an automated email from the ASF dual-hosted git repository.

aradzinski pushed a commit to branch NLPCRAFT-287
in repository https://gitbox.apache.org/repos/asf/incubator-nlpcraft.git


The following commit(s) were added to refs/heads/NLPCRAFT-287 by this push:
     new 83b3b6a  Code review.
     new 1900300  Merge branch 'NLPCRAFT-287' of 
https://github.com/apache/incubator-nlpcraft into NLPCRAFT-287
83b3b6a is described below

commit 83b3b6a349f6651ce94b8bc31081de2a181701f7
Author: Aaron Radzinzski <[email protected]>
AuthorDate: Fri Apr 16 00:54:22 2021 +0300

    Code review.
---
 .../scala/org/apache/nlpcraft/probe/mgrs/NCProbeSynonym.scala     | 4 +---
 .../org/apache/nlpcraft/probe/mgrs/model/NCModelManager.scala     | 8 ++++----
 .../nlpcraft/probe/mgrs/nlp/enrichers/model/NCModelEnricher.scala | 1 -
 3 files changed, 5 insertions(+), 8 deletions(-)

diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/NCProbeSynonym.scala 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/NCProbeSynonym.scala
index d5361f8..d3d1e5a 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/NCProbeSynonym.scala
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/NCProbeSynonym.scala
@@ -198,9 +198,7 @@ class NCProbeSynonym(
         else
             false
     }
-
-
-
+    
     /**
       *
       * @param toks
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/model/NCModelManager.scala
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/model/NCModelManager.scala
index 5c4ca4c..e2e0833 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/model/NCModelManager.scala
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/model/NCModelManager.scala
@@ -73,10 +73,10 @@ object NCModelManager extends NCService with 
DecorateAsScala {
                     s"Origin:                ${mdl.getOrigin}",
                     s"Elements:              
${withWarn(w.elements.keySet.size)}",
                     s"Synonyms:",
-                    s" - Simple, continuous: $contCnt",
-                    s" - Simple, sparse:     $sparseCnt",
-                    s" - IDL, continuous:    $contIdlCnt",
-                    s" - IDL, sparse:        $sparseIdlCnt",
+                    s"   Simple continuous:  $contCnt",
+                    s"   Simple sparse:      $sparseCnt",
+                    s"   IDL continuous:     $contIdlCnt",
+                    s"   IDL sparse:         $sparseIdlCnt",
                     s"Intents:               ${withWarn(w.intents.size)}"
                 )
             })
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/nlp/enrichers/model/NCModelEnricher.scala
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/nlp/enrichers/model/NCModelEnricher.scala
index 52a84f0..1bea582 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/nlp/enrichers/model/NCModelEnricher.scala
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/probe/mgrs/nlp/enrichers/model/NCModelEnricher.scala
@@ -406,7 +406,6 @@ object NCModelEnricher extends NCProbeEnricher with 
DecorateAsScala {
 
         h.complexes.par.
             flatMap(complexSeq ⇒ {
-                //val rec = 
complexSeq.tokensComplexes.filter(_.isSubsetOf(idxMin, idxMax, idxs))
                 val rec = 
complexSeq.tokensComplexes.filter(_.wordIndexes.exists(idxs.contains))
 
                 // Drops without tokens (IDL part works with tokens).

Reply via email to