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: [email protected] With regards, Apache Git Services
