rlworkgroup / dowel

A little logger for machine learning research
MIT License
32 stars 37 forks source link

Replace logger.tabular with Logger.log_kv #8

Open ryanjulian opened 5 years ago

ryanjulian commented 5 years ago

Logger.TabularInput is needlessly cumbersome for the most common logging use case, and forces LogOutputs to replicate parts of the logger's dispatch and accounting logic internally. It would be simpler and more consistent just to provide a key-based API.

This would also open up the logger to features like filtering output handlers based on key regexs.