BF - remove unused log_level, make error_level functional

Author: matthew-brett

nibabel/analyze.py

@@ -680,10 +680,12 @@ def items(self):
         ''' Return items from header data'''
         return zip(self.keys(), self.values())
 
-    def check_fix(self,
-              logger=imageglobals.logger,
-              error_level=imageglobals.error_level):
+    def check_fix(self, logger=None, error_level=None):
         ''' Check header data with checks '''
+        if logger is None:
+            logger = imageglobals.logger
+        if error_level is None:
+            error_level = imageglobals.error_level
         battrun = BatteryRunner(self.__class__._get_checks())
         self, reports = battrun.check_fix(self)
         for report in reports:

nibabel/imageglobals.py

@@ -6,11 +6,21 @@
 #   copyright and license terms.
 #
 ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
-''' Defaults for images and headers '''
+""" Defaults for images and headers
+
+error_level is the problem level (see BatteryRunners) at which an error will be
+raised, by the batteryrunners ``log_raise`` method.  Thus a level of 0 will
+result in an error for any problem at all, and a level of 50 will mean no errors
+will be raised (unless someone's put some strange problem_level > 50 code in).
+
+``logger`` is the default logger (python log instance)
+
+To set the log level (log message appears for problem of level >= log level),
+use e.g. ``logger.level = 40``
+"""
 
 import logging
 
 error_level = 40
-log_level = 30
 logger = logging.getLogger('nifti.global')
 logger.addHandler(logging.StreamHandler())