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

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


The following commit(s) were added to refs/heads/NLPCRAFT-468 by this push:
     new 612b9d5  WIP.
612b9d5 is described below

commit 612b9d514304179890d875ba955e700857f98392
Author: Sergey Kamov <[email protected]>
AuthorDate: Tue Oct 12 20:36:31 2021 +0300

    WIP.
---
 .../scala/org/apache/nlpcraft/model/NCConversation.java |  3 ++-
 .../scala/org/apache/nlpcraft/model/NCModelConfig.java  | 11 +++++------
 .../detectors/NCDefaultStopWordsDetector.java           |  4 +---
 .../detectors/NCDefaultSwearWordsDetector.java          |  4 +---
 .../detectors/NCFileWordsDetector.java                  |  2 +-
 .../ner/opennlp/NCOpenNlpNerParser.java                 |  2 +-
 .../ner/synonyms/NCSynonymsNerElement.java              |  2 +-
 .../ner/synonyms/NCSynonymsNerParser.java               |  2 +-
 .../ner/synonyms/NCSynonymsNerValue.java                |  2 +-
 .../ner/synonyms/NCSynonymsNerValueLoader.java          |  2 +-
 .../synonyms/builders/NCSynonymsNerElementBuilder.java  |  8 ++++----
 .../synonyms/builders/NCSynonymsNerParserBuilder.java   |  6 +++---
 .../tokenizer/NCOpenNlpTokenizer.java                   |  2 +-
 .../scala/org/apache/nlpcraft/model/nlp/NCNlpToken.java |  4 ++--
 nlpcraft/src/test/java/org/apache/nlpcraft/NCSpec.java  | 17 ++++++++---------
 15 files changed, 33 insertions(+), 38 deletions(-)

diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCConversation.java 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCConversation.java
index f4f28dd..fd30366 100644
--- a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCConversation.java
+++ b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCConversation.java
@@ -17,12 +17,13 @@
 
 package org.apache.nlpcraft.model;
 
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
 
 import java.util.List;
 import java.util.function.Predicate;
 
 /**
+ * TODO: drop reference to NCSynonymsNerElement.
  * Conversation container for unique combination of user and data model.
  * <p>
  * Conversation management is based on idea of a short-term-memory (STM). STM 
can be viewed as a condensed
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCModelConfig.java 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCModelConfig.java
index 0dd105c..2e66e17 100644
--- a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCModelConfig.java
+++ b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/NCModelConfig.java
@@ -17,11 +17,10 @@
 
 package org.apache.nlpcraft.model;
 
-import org.apache.nlpcraft.model.impl.detectors.NCDefaultStopWordsDetector;
-import org.apache.nlpcraft.model.impl.detectors.NCDefaultSwearWordsDetector;
-import org.apache.nlpcraft.model.impl.detectors.NCFileWordsDetector;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
-import org.apache.nlpcraft.model.impl.tokenizer.NCOpenNlpTokenizer;
+import 
org.apache.nlpcraft.model.components.detectors.NCDefaultStopWordsDetector;
+import 
org.apache.nlpcraft.model.components.detectors.NCDefaultSwearWordsDetector;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
+import org.apache.nlpcraft.model.components.tokenizer.NCOpenNlpTokenizer;
 import org.apache.nlpcraft.model.nlp.NCNlpNerParser;
 import org.apache.nlpcraft.model.nlp.NCNlpTokenizer;
 import org.apache.nlpcraft.model.nlp.NCNlpWordsDetector;
@@ -31,7 +30,6 @@ import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 
 /**
  * Read-only view on data model. Model view defines a declarative, or 
configurable, part of the model.
@@ -163,6 +161,7 @@ public interface NCModelConfig {
     int MODEL_VERSION_MAXLEN = 16;
 
     /**
+     * TODO: javadoc
      * Max length for {@link NCSynonymsNerElement#getId()} method.
      */
     int MODEL_ELEMENT_ID_MAXLEN = 64;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultStopWordsDetector.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultStopWordsDetector.java
similarity index 92%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultStopWordsDetector.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultStopWordsDetector.java
index baa99bb..5ab3c26 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultStopWordsDetector.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultStopWordsDetector.java
@@ -15,13 +15,11 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.detectors;
+package org.apache.nlpcraft.model.components.detectors;
 
 import org.apache.nlpcraft.model.nlp.NCNlpWord;
 import org.apache.nlpcraft.model.nlp.NCNlpWordsDetector;
 
-import java.io.File;
-import java.net.URL;
 import java.util.Collections;
 import java.util.List;
 
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultSwearWordsDetector.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultSwearWordsDetector.java
similarity index 92%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultSwearWordsDetector.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultSwearWordsDetector.java
index 1669e65..44abf59 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCDefaultSwearWordsDetector.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCDefaultSwearWordsDetector.java
@@ -15,13 +15,11 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.detectors;
+package org.apache.nlpcraft.model.components.detectors;
 
 import org.apache.nlpcraft.model.nlp.NCNlpWord;
 import org.apache.nlpcraft.model.nlp.NCNlpWordsDetector;
 
-import java.io.File;
-import java.net.URL;
 import java.util.Collections;
 import java.util.List;
 
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCFileWordsDetector.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCFileWordsDetector.java
similarity index 95%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCFileWordsDetector.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCFileWordsDetector.java
index 3043843..53765f3 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/detectors/NCFileWordsDetector.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/detectors/NCFileWordsDetector.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.detectors;
+package org.apache.nlpcraft.model.components.detectors;
 
 import org.apache.nlpcraft.model.nlp.NCNlpWord;
 import org.apache.nlpcraft.model.nlp.NCNlpWordsDetector;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/opennlp/NCOpenNlpNerParser.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/opennlp/NCOpenNlpNerParser.java
similarity index 95%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/opennlp/NCOpenNlpNerParser.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/opennlp/NCOpenNlpNerParser.java
index d2fbb5b..e030b83 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/opennlp/NCOpenNlpNerParser.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/opennlp/NCOpenNlpNerParser.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.opennlp;
+package org.apache.nlpcraft.model.components.ner.opennlp;
 
 import org.apache.nlpcraft.model.NCModelConfig;
 import org.apache.nlpcraft.model.NCRequest;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerElement.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerElement.java
similarity index 99%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerElement.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerElement.java
index 0c34c05..58ab8e7 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerElement.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerElement.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms;
+package org.apache.nlpcraft.model.components.ner.synonyms;
 
 import org.apache.nlpcraft.model.NCConversation;
 import org.apache.nlpcraft.model.NCToken;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerParser.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerParser.java
similarity index 99%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerParser.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerParser.java
index 89214c0..ac83b7d 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerParser.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerParser.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms;
+package org.apache.nlpcraft.model.components.ner.synonyms;
 
 import org.apache.nlpcraft.model.NCModel;
 import org.apache.nlpcraft.model.nlp.NCNlpNerParser;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValue.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValue.java
similarity index 96%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValue.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValue.java
index 075b7e5..b1b6ebb 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValue.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValue.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms;
+package org.apache.nlpcraft.model.components.ner.synonyms;
 
 import java.util.List;
 
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValueLoader.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValueLoader.java
similarity index 97%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValueLoader.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValueLoader.java
index 0744e7a..a4ea345 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/NCSynonymsNerValueLoader.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/NCSynonymsNerValueLoader.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms;
+package org.apache.nlpcraft.model.components.ner.synonyms;
 
 import java.util.Set;
 
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
similarity index 85%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
index 836d9aa..f0dd2b5 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerElementBuilder.java
@@ -15,11 +15,11 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms.builders;
+package org.apache.nlpcraft.model.components.ner.synonyms.builders;
 
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerValue;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerValueLoader;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerValue;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
+import 
org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerValueLoader;
 
 import java.util.List;
 import java.util.Map;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
similarity index 88%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
index bf95154..a5f54ae 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/ner/synonyms/builders/NCSynonymsNerParserBuilder.java
@@ -15,10 +15,10 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.ner.synonyms.builders;
+package org.apache.nlpcraft.model.components.ner.synonyms.builders;
 
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerParser;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerParser;
 
 import java.io.File;
 import java.util.List;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/tokenizer/NCOpenNlpTokenizer.java
 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/tokenizer/NCOpenNlpTokenizer.java
similarity index 95%
rename from 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/tokenizer/NCOpenNlpTokenizer.java
rename to 
nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/tokenizer/NCOpenNlpTokenizer.java
index 7cccfd8..43df0e4 100644
--- 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/impl/tokenizer/NCOpenNlpTokenizer.java
+++ 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/components/tokenizer/NCOpenNlpTokenizer.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.nlpcraft.model.impl.tokenizer;
+package org.apache.nlpcraft.model.components.tokenizer;
 
 import org.apache.nlpcraft.model.NCRequest;
 import org.apache.nlpcraft.model.nlp.NCNlpTokenizer;
diff --git 
a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/nlp/NCNlpToken.java 
b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/nlp/NCNlpToken.java
index 0d182e0..e6888f9 100644
--- a/nlpcraft/src/main/scala/org/apache/nlpcraft/model/nlp/NCNlpToken.java
+++ b/nlpcraft/src/main/scala/org/apache/nlpcraft/model/nlp/NCNlpToken.java
@@ -20,7 +20,7 @@ package org.apache.nlpcraft.model.nlp;
 import org.apache.nlpcraft.model.NCModel;
 import org.apache.nlpcraft.model.NCModelConfig;
 import org.apache.nlpcraft.model.NCRequest;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
 
 import java.util.Collections;
 import java.util.List;
@@ -34,7 +34,7 @@ public interface NCNlpToken {
      * references the element that's already defined in the model.
      *
      * @return ID of the detected model element.
-     * @see NCSynonymsNerElement#getId()
+     * @see NCSynonymsNerElement#getId() TODO: drop it.
      * @see NCModel#getElements()
      */
     String getId();
diff --git a/nlpcraft/src/test/java/org/apache/nlpcraft/NCSpec.java 
b/nlpcraft/src/test/java/org/apache/nlpcraft/NCSpec.java
index d8b9eaf..48bfaea 100644
--- a/nlpcraft/src/test/java/org/apache/nlpcraft/NCSpec.java
+++ b/nlpcraft/src/test/java/org/apache/nlpcraft/NCSpec.java
@@ -24,15 +24,15 @@ import org.apache.nlpcraft.model.NCRejection;
 import org.apache.nlpcraft.model.NCModelConfig;
 import org.apache.nlpcraft.model.NCResult;
 import org.apache.nlpcraft.model.builders.NCModelConfigBuilder;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerValue;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerValue;
 import org.apache.nlpcraft.model.builders.NCModelBuilder;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerElement;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerParser;
-import org.apache.nlpcraft.model.impl.ner.synonyms.NCSynonymsNerValueLoader;
-import 
org.apache.nlpcraft.model.impl.ner.synonyms.builders.NCSynonymsNerElementBuilder;
-import 
org.apache.nlpcraft.model.impl.ner.synonyms.builders.NCSynonymsNerParserBuilder;
-import org.apache.nlpcraft.model.impl.ner.opennlp.NCOpenNlpNerParser;
-import org.apache.nlpcraft.model.impl.tokenizer.NCOpenNlpTokenizer;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerElement;
+import org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerParser;
+import 
org.apache.nlpcraft.model.components.ner.synonyms.NCSynonymsNerValueLoader;
+import 
org.apache.nlpcraft.model.components.ner.synonyms.builders.NCSynonymsNerElementBuilder;
+import 
org.apache.nlpcraft.model.components.ner.synonyms.builders.NCSynonymsNerParserBuilder;
+import org.apache.nlpcraft.model.components.ner.opennlp.NCOpenNlpNerParser;
+import org.apache.nlpcraft.model.components.tokenizer.NCOpenNlpTokenizer;
 import org.junit.jupiter.api.Test;
 
 import java.io.File;
@@ -40,7 +40,6 @@ import java.net.URL;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.Set;
 import org.apache.nlpcraft.model.NCIntentRef;
 

Reply via email to