[opennlp] branch main updated: OPENNLP-1449 - Revise log levels in OpenNLP

[rzo1]

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

rzo1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/opennlp.git


The following commit(s) were added to refs/heads/main by this push:
     new e924e711 OPENNLP-1449 - Revise log levels in OpenNLP
e924e711 is described below

commit e924e711b67fb71586e388a5e687014cb2842035
Author: Richard Zowalla <richard.zowa...@hs-heilbronn.de>
AuthorDate: Tue Feb 28 08:55:56 2023 +0100

    OPENNLP-1449 - Revise log levels in OpenNLP
---
 .../src/main/java/opennlp/tools/formats/masc/MascDocument.java        | 2 +-
 .../src/main/java/opennlp/tools/formats/masc/MascDocumentStream.java  | 2 +-
 .../opennlp/tools/ml/maxent/quasinewton/ParallelNegLogLikelihood.java | 2 +-
 .../src/main/java/opennlp/tools/ml/perceptron/PerceptronTrainer.java  | 2 +-
 .../src/main/java/opennlp/tools/tokenize/TokSpanEventStream.java      | 2 +-
 .../main/java/opennlp/tools/tokenize/lang/en/TokenSampleStream.java   | 4 ++--
 6 files changed, 7 insertions(+), 7 deletions(-)

diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocument.java 
b/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocument.java
index 6a31e63b..aec39b88 100644
--- a/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocument.java
+++ b/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocument.java
@@ -353,7 +353,7 @@ public class MascDocument {
       //Check that all tokens have at least one quark.
       for (Map.Entry<Integer, int[]> token : tokenToQuarks.entrySet()) {
         if (token.getValue().length == 0) {
-          logger.error("Token without quarks: {}", token.getKey());
+          logger.warn("Token without quarks: {}", token.getKey());
         }
       }
 
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocumentStream.java
 
b/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocumentStream.java
index a526d826..afe02a2f 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocumentStream.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/formats/masc/MascDocumentStream.java
@@ -143,7 +143,7 @@ public class MascDocumentStream implements 
ObjectStream<MascDocument> {
               documents.add(MascDocument.parseDocument(hdrFilePath, f_primary, 
f_seg,
                   f_penn, f_s, f_ne));
             } catch (IOException e) {
-              logger.warn("Failed to parse the file: {}", hdrFilePath, e);
+              logger.error("Failed to parse the file: {}", hdrFilePath, e);
               failedLoads++;
             }
           }
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/ml/maxent/quasinewton/ParallelNegLogLikelihood.java
 
b/opennlp-tools/src/main/java/opennlp/tools/ml/maxent/quasinewton/ParallelNegLogLikelihood.java
index 740e3ac7..b681e0b3 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/ml/maxent/quasinewton/ParallelNegLogLikelihood.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/ml/maxent/quasinewton/ParallelNegLogLikelihood.java
@@ -154,7 +154,7 @@ public class ParallelNegLogLikelihood extends 
NegLogLikelihood {
         future.get();
 
     } catch (Exception e) {
-      logger.warn(e.getLocalizedMessage(), e);
+      logger.error(e.getLocalizedMessage(), e);
     }
 
     executor.shutdown();
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/PerceptronTrainer.java
 
b/opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/PerceptronTrainer.java
index b9cddc96..77a9db1e 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/PerceptronTrainer.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/PerceptronTrainer.java
@@ -394,7 +394,7 @@ public class PerceptronTrainer extends AbstractEventTrainer 
{
       if (StrictMath.abs(prevAccuracy1 - trainingAccuracy) < tolerance
           && StrictMath.abs(prevAccuracy2 - trainingAccuracy) < tolerance
           && StrictMath.abs(prevAccuracy3 - trainingAccuracy) < tolerance) {
-        logger.info("Stopping: change in training set accuracy less than {}", 
tolerance);
+        logger.warn("Stopping: change in training set accuracy less than {}", 
tolerance);
         break;
       }
 
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/tokenize/TokSpanEventStream.java 
b/opennlp-tools/src/main/java/opennlp/tools/tokenize/TokSpanEventStream.java
index 6e534f6a..0f693262 100644
--- a/opennlp-tools/src/main/java/opennlp/tools/tokenize/TokSpanEventStream.java
+++ b/opennlp-tools/src/main/java/opennlp/tools/tokenize/TokSpanEventStream.java
@@ -138,7 +138,7 @@ public class TokSpanEventStream extends 
AbstractEventStream<TokenSample> {
               //keep looking
             }
             else {
-              logger.info("Bad training token: " + tokens[ti] + " cand: " + 
cSpan +
+              logger.warn("Bad training token: " + tokens[ti] + " cand: " + 
cSpan +
                   " token=" + text.substring(tokens[ti].getStart(), 
tokens[ti].getEnd()));
             }
           }
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/tokenize/lang/en/TokenSampleStream.java
 
b/opennlp-tools/src/main/java/opennlp/tools/tokenize/lang/en/TokenSampleStream.java
index eb6fc087..7884f681 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/tokenize/lang/en/TokenSampleStream.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/tokenize/lang/en/TokenSampleStream.java
@@ -121,7 +121,7 @@ public class TokenSampleStream implements 
Iterator<TokenSample> {
   }
 
   private static void usage() {
-    logger.error("TokenSampleStream [-spans] < in");
-    logger.error("Where in is a space delimited list of tokens.");
+    logger.info("TokenSampleStream [-spans] < in");
+    logger.info("Where in is a space delimited list of tokens.");
   }
 }