From ed0eed59b25dbc05b429749c15027a412e81776e Mon Sep 17 00:00:00 2001 From: Yong-Bin Zhuang <38876805+robinzyb@users.noreply.github.com> Date: Thu, 18 Dec 2025 21:53:57 +0900 Subject: [PATCH 1/2] Refactor logging setup in log.py --- cp2kdata/log.py | 28 ++++++++++++++++++++-------- 1 file changed, 20 insertions(+), 8 deletions(-) diff --git a/cp2kdata/log.py b/cp2kdata/log.py index 928d4c0..8f4c058 100644 --- a/cp2kdata/log.py +++ b/cp2kdata/log.py @@ -4,13 +4,25 @@ level_name = os.environ.get('CP2KDATA_LOG_LEVEL', 'INFO') level = logging._nameToLevel.get(level_name, logging.INFO) -# format to include timestamp and module -if level_name == 'DEBUG': - logging.basicConfig(format='CP2KDATA| %(asctime)s - %(levelname)-8s %(name)-40s: %(message)s', level=level) -else: - logging.basicConfig(format='CP2KDATA| %(message)s', level=level) -# suppress transitions logging -# logging.getLogger('transitions.core').setLevel(logging.WARNING) +logger = logging.getLogger(_LOGGER_NAME) +logger.setLevel(level) +logger.propagate = False # do NOT touch root logger + +if not logger.handlers: + handler = logging.StreamHandler() + + if level_name == "DEBUG": + formatter = logging.Formatter( + "CP2KDATA| %(asctime)s - %(levelname)-8s %(name)-40s: %(message)s" + ) + else: + formatter = logging.Formatter("CP2KDATA| %(message)s") + + handler.setFormatter(formatter) + logger.addHandler(handler) + def get_logger(name=None): - return logging.getLogger(name) \ No newline at end of file + if name: + return logging.getLogger(f"{_LOGGER_NAME}.{name}") + return logger From 372e89a016e50b1fd311db2f3e58220362183f47 Mon Sep 17 00:00:00 2001 From: Yong-Bin Zhuang <38876805+robinzyb@users.noreply.github.com> Date: Thu, 18 Dec 2025 22:00:19 +0900 Subject: [PATCH 2/2] Add logger configuration to log.py --- cp2kdata/log.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cp2kdata/log.py b/cp2kdata/log.py index 8f4c058..352560b 100644 --- a/cp2kdata/log.py +++ b/cp2kdata/log.py @@ -1,6 +1,8 @@ import logging import os +_LOGGER_NAME = "cp2kdata" + level_name = os.environ.get('CP2KDATA_LOG_LEVEL', 'INFO') level = logging._nameToLevel.get(level_name, logging.INFO)