Fix experiment_config error message (#257)

comet-ml · Sep 16, 2024 · 6557b8a · 6557b8a
1 parent 46012fa
commit 6557b8a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/sdks/python/src/opik/api_objects/opik_client.py b/sdks/python/src/opik/api_objects/opik_client.py
@@ -363,9 +363,10 @@ def create_experiment(
 
         if isinstance(experiment_config, Mapping):
             metadata = jsonable_encoder.jsonable_encoder(experiment_config)
-        else:
+        elif experiment_config is not None:
             LOGGER.error(
-                "Experiment config must be dictionary, but %s was provided. Config will not be logged."
+                "Experiment config must be dictionary, but %s was provided. Config will not be logged.",
+                experiment_config,
             )
             metadata = None
 

diff --git a/sdks/python/tests/e2e/conftest.py b/sdks/python/tests/e2e/conftest.py
@@ -15,7 +15,6 @@ def _random_chars(n: int = 6) -> str:
 @pytest.fixture(scope="session")
 def configure_e2e_tests_env():
     os.environ["OPIK_PROJECT_NAME"] = "e2e-tests"
-    os.environ["OPIK_URL_OVERRIDE"] = "http://localhost:5173/api"
 
 
 @pytest.fixture()