Merge pull request #208 from tornede/204-table_name-parameter-does-not-adapt-name-of-logtable

204 table name parameter does not adapt name of logtable

Author: LukasFehring

.gitignore

@@ -150,6 +150,8 @@ output/
 # codecarbon
 .codecarbon.config
 emissions.csv
+powermetrics_log.txt
 
 # development folder
 development/
+

CHANGELOG.rst

@@ -1,7 +1,13 @@
 =========
 Changelog
 =========
+v1.4.3 (**.**.2025)
+===================
+
+Fix
+===
 
+- Fix bug, where the logtable_name was not overwritten by `table_name` updates in the `PyExperimenter` class.
 
 v1.4.2 (12.06.2024)
 ===================

docs/source/examples/example_logtables.ipynb

@@ -1,15 +1,18 @@
 import logging
 from abc import ABC, abstractclassmethod
 from logging import Logger
-from typing import Any, Dict, List, Tuple, Union
+from typing import Any, Dict, List, Optional, Tuple, Union
 
 import numpy as np
 import omegaconf
 from attr import dataclass
 from omegaconf import DictConfig, ListConfig, OmegaConf
 
 from py_experimenter import utils
-from py_experimenter.exceptions import InvalidColumnError, InvalidConfigError, InvalidLogtableError
+from py_experimenter.exceptions import (
+    InvalidColumnError,
+    InvalidLogtableError,
+)
 
 
 class Cfg(ABC):
@@ -362,8 +365,10 @@ def __init__(
         self.codecarbon_configuration = codecarbon_configuration
 
     @staticmethod
-    def extract_config(config_path: str, logger: logging.Logger) -> "PyExperimenterCfg":
+    def extract_config(config_path: str, logger: logging.Logger, overwritten_table_name: Optional[str] = None) -> "PyExperimenterCfg":
         config = omegaconf.OmegaConf.load(config_path)
+        if overwritten_table_name is not None:
+            config["PY_EXPERIMENTER"]["Database"]["table"]["name"] = overwritten_table_name
 
         if "n_jobs" not in config["PY_EXPERIMENTER"]:
             config["PY_EXPERIMENTER"]["n_jobs"] = 1

poetry.lock

@@ -96,7 +96,7 @@ def __init__(
             handler.setFormatter(formatter)
             self.logger.addHandler(handler)
 
-        self.config = PyExperimenterCfg.extract_config(experiment_configuration_file_path, logger=self.logger)
+        self.config = PyExperimenterCfg.extract_config(experiment_configuration_file_path, logger=self.logger, overwritten_table_name=table_name)
 
         self.use_codecarbon = use_codecarbon
 
@@ -109,8 +109,6 @@ def __init__(
         if use_ssh_tunnel is not None:
             self.config.database_configuration.use_ssh_tunnel = use_ssh_tunnel
 
-        if table_name is not None:
-            self.config.database_configuration.table_name = table_name
         if database_name is not None:
             self.config.database_configuration.database_name = database_name
         self.name = name

py_experimenter/config.py

@@ -37,6 +37,7 @@ codecarbon = ">=2.2.1"
 pymysql = "^1.0.3"
 omegaconf = "^2.3.0"
 sshtunnel = "^0.4.0"
+setuptools = "^69.5.1"
 
 [tool.poetry.group.dev.dependencies]
 pytest = ">=7.0"