Minor fix in log, extend default config

[moe.git] / t / moe / log.py

diff --git a/t/moe/log.py b/t/moe/log.py
index 1660a028b70441a23595750f988aa8d30136a20f..aea63cb2c7afc16d8aefe73a719ecd240f1225dc 100644 (file)
--- a/t/moe/log.py
+++ b/t/moe/log.py
@@ -1,8 +1,23 @@
 #!/usr/bin/env python
 
+"""
+
+`Loggers` is a collection of logggers for :class:`~moe.eval.Eval`, initializing 4 subloggers (see class description).
+
+Use as `e.log.debug(...)` for main log that goes both to `test` and `main` log,
+`e.log.test.debug(...)` for individual logs.
+
+.. :data:: DDEBUG = 5
+       Very verbose debugging level
+
+"""
+
+
 import sys, os
 import logging 
-from logging import Logger, StreamHandler
+from logging import Logger, StreamHandler, Formatter
+
+logging.addLevelName(5,'DDEBUG')
 
 
 class Loggers(Logger):
@@ -18,7 +33,7 @@ class Loggers(Logger):
     per-test log, initially no handler, to be directed to file like `{TEST}.log`
 
   .. note:: Currently, the logs and fd's get never closed. 
-  .. note:: Currently, the `Loggers.open_eval_log` changes fd's 1 and 2 by default, this is a global change.
+  .. warning:: `Loggers.open_eval_log` changes global fd's 1 and 2 by default.
   """
   def __init__(self):
     "Initialize loggers as described in class description."
@@ -37,7 +52,7 @@ class Loggers(Logger):
     self.test_file = None
     # user progress logger  
     self.user = Logger('user')
-    self.eval.addHandler(StreamHandler(self.orig_stdout_file))
+    self.user.addHandler(StreamHandler(self.orig_stdout_file))
  
     self.addHandler(self.test)
     self.addHandler(self.eval)
@@ -82,7 +97,7 @@ class Loggers(Logger):
   def close_test_log(self):
     """Close per-test logfile, leaving only the null handler."""
     if self.test_handler:
-      self.test.info('Closing logfile' % filename)
+      self.test.info('Closing test logfile')
       self.test.removeHandler(self.test_handler)
     self.test_handler = None
     self.test_file.close()