WIP: remove delegation

Author: braingram

src/stpipe/log.py

@@ -291,7 +291,7 @@ def record_logs(level=logging.NOTSET, formatter=None):
     else:
         handler = RecordingHandler(level=level)
         handler.setFormatter(formatter)
-        logger = getLogger(STPIPE_ROOT_LOGGER)
+        logger = getLogger()
         logger.addHandler(handler)
         try:
             yield handler.log_records
@@ -303,9 +303,9 @@ def record_logs(level=logging.NOTSET, formatter=None):
 # uses the `delegator` instance to change what the current Step logger
 # is.
 log = getLogger()
-delegator = DelegationHandler()
-delegator.log = getLogger(STPIPE_ROOT_LOGGER)
-log.addHandler(delegator)
+#delegator = DelegationHandler()
+#delegator.log = getLogger(STPIPE_ROOT_LOGGER)
+#log.addHandler(delegator)
 
 logging_config_file = _find_logging_config_file()
 if logging_config_file is not None:

src/stpipe/pipeline.py

@@ -3,6 +3,7 @@
 """
 
 from collections.abc import Sequence
+import logging
 from os.path import dirname, join
 from typing import ClassVar
 
@@ -15,7 +16,8 @@
 
 # For classmethods, the logger to use is the
 # delegator, since the pipeline has not yet been instantiated.
-logger = log.delegator.log
+#logger = log.delegator.log
+logger = logging.getLogger(log.STPIPE_ROOT_LOGGER)
 
 
 class Pipeline(Step):

src/stpipe/step.py

@@ -426,8 +426,8 @@ def run(self, *args):
             self._log_records = log_records
 
             # Make generic log messages go to this step's logger
-            orig_log = log.delegator.log
-            log.delegator.log = self.log
+            #orig_log = log.delegator.log
+            #log.delegator.log = self.log
 
             step_result = None
 
@@ -572,7 +572,8 @@ def set_skipped(model):
                 if not self.skip:
                     self.log.info("Step %s done", self.name)
             finally:
-                log.delegator.log = orig_log
+                #log.delegator.log = orig_log
+                pass
 
         return step_result