[ 
https://issues.apache.org/jira/browse/OPENNLP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17693149#comment-17693149
 ] 

ASF GitHub Bot commented on OPENNLP-1448:
-----------------------------------------

rzo1 commented on code in PR #492:
URL: https://github.com/apache/opennlp/pull/492#discussion_r1116846141


##########
opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/SimplePerceptronSequenceTrainer.java:
##########
@@ -195,11 +199,12 @@ public AbstractModel trainModel(int iterations, 
SequenceStream<Event> sequenceSt
       updates = new int[numPreds][numOutcomes][3];
     }
 
-    display("done.\n");
+    logger.info("done.");
+
+    logger.info("Number of Event Tokens: {}", numEvents);

Review Comment:
   done





> Introduce SLF4J in OpenNLP
> --------------------------
>
>                 Key: OPENNLP-1448
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1448
>             Project: OpenNLP
>          Issue Type: Sub-task
>            Reporter: Richard Zowalla
>            Assignee: Richard Zowalla
>            Priority: Major
>
> This will be the first step regarding OPENNLP-1447.
> Goal is to replace System.err / System.out calls with logger output, which is 
> configurable.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to