[ https://issues.apache.org/jira/browse/OPENNLP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17692849#comment-17692849 ]
ASF GitHub Bot commented on OPENNLP-1448: ----------------------------------------- rzo1 commented on code in PR #492: URL: https://github.com/apache/opennlp/pull/492#discussion_r1116135459 ########## opennlp-tools/src/main/java/opennlp/tools/ml/perceptron/SimplePerceptronSequenceTrainer.java: ########## @@ -372,14 +362,11 @@ public void nextIteration(int iteration) throws IOException { if (predParams[oi] != 0) { predParams[oi] /= totIterations; averageParams[pi].setParameter(oi, predParams[oi]); - //System.err.println("updates["+pi+"]["+oi+"]=("+updates[pi][oi][ITER]+"," - // +updates[pi][oi][EVENT]+","+updates[pi][oi][VALUE]+") + ("+iterations+","+0+"," - // +params[pi].getParameters()[oi]+") -> "+averageParams[pi].getParameters()[oi]); } Review Comment: Trace? Similar to @mawiesne comments below? > 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)