liuzh91 commented on issue #16922: Update LoggingHandler to support logging per interval
URL: https://github.com/apache/incubator-mxnet/pull/16922#issuecomment-560980189
 
 
   > I'm actually not sure if logging both `self.train_metrics + self.val_metrics` during `LOG_PER_INTERVAL` is good.
   > Currently the notion of `val_metrics` and `train_metrics` is not clearly decoupled. Thinking about how to tackle #16959 may also help to clarify the relation of `eval_metrics` and `train_metrics` and how their values should be updated and logged.
   
   It makes sense to me. I'll merge `LOG_PER_BATCH` and `LOG_PER_INTERVAL` and leave out the `val_metrics`. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services