merge LogTrainingHook and LogSessionHook?

Open brandondutra opened this issue 8 years ago • 3 comments

LogSessionHook: shows time of last batch train, running total time, running throughput

LogTrainingHook: loss of last batch train, running total time, running throughput. Also makes summary events

Question: why do we run/have both? I basically see the same running total time, running throughput twice.

Mar 01 '17 07:03 brandondutra

PR sent to remove the duplicated logging.

Mar 08 '17 22:03 brandondutra

Lets consider WorkerLoggingHook and MasterLoggingHook where each generates a compact single log line -- goal is to show progress on each node in the cluster, with least possible impact on log.

Mar 09 '17 04:03 nikhilk

See #13 for basic first pr.

Mar 10 '17 18:03 brandondutra