replace config.cache.dir with config.cache_dir

Author: geetu040

openml/_api/__init__.py

@@ -41,7 +41,6 @@
     APIBackend,
     APIBackendBuilder,
     APIConfig,
-    CacheConfig,
     Config,
     ConnectionConfig,
     _backend,
@@ -52,7 +51,6 @@
     "APIBackend",
     "APIBackendBuilder",
     "APIConfig",
-    "CacheConfig",
     "Config",
     "ConnectionConfig",
     "DatasetAPI",

openml/_api/setup/__init__.py

@@ -1,13 +1,12 @@
 from ._instance import _backend
 from .backend import APIBackend
 from .builder import APIBackendBuilder
-from .config import APIConfig, CacheConfig, Config, ConnectionConfig
+from .config import APIConfig, Config, ConnectionConfig
 
 __all__ = [
     "APIBackend",
     "APIBackendBuilder",
     "APIConfig",
-    "CacheConfig",
     "Config",
     "ConnectionConfig",
     "_backend",

openml/_api/setup/builder.py

@@ -82,7 +82,7 @@ def build(cls, config: Config) -> APIBackendBuilder:
         APIBackendBuilder
             Builder instance with all resource API interfaces initialized.
         """
-        cache_dir = Path(config.cache.dir).expanduser()
+        cache_dir = Path(config.cache_dir).expanduser()
 
         http_cache = HTTPCache(path=cache_dir)
         minio_client = MinIOClient(path=cache_dir)

openml/_api/setup/config.py

@@ -44,20 +44,6 @@ class ConnectionConfig:
     retry_policy: RetryPolicy
 
 
-@dataclass
-class CacheConfig:
-    """
-    Configuration for caching API responses locally.
-
-    Parameters
-    ----------
-    dir : str
-        Path to the directory where cached files will be stored.
-    """
-
-    dir: str
-
-
 @dataclass
 class Config:
     """
@@ -71,16 +57,17 @@ class Config:
         Primary API version to use (default is V1).
     fallback_api_version : APIVersion or None
         Optional fallback API version if the primary API does not support certain operations.
+    cache_dir : str
+        Path to the directory where cached files will be stored.
     api_configs : dict of APIVersion to APIConfig
         Mapping from API version to its server/base URL and API key configuration.
     connection : ConnectionConfig
         Settings for request retries and retry policy.
-    cache : CacheConfig
-        Settings for local caching of API responses.
     """
 
     api_version: APIVersion = APIVersion.V1
     fallback_api_version: APIVersion | None = None
+    cache_dir: str = str(_resolve_default_cache_dir())
 
     api_configs: dict[APIVersion, APIConfig] = field(
         default_factory=lambda: {
@@ -103,9 +90,3 @@ class Config:
             retry_policy=RetryPolicy.HUMAN,
         )
     )
-
-    cache: CacheConfig = field(
-        default_factory=lambda: CacheConfig(
-            dir=str(_resolve_default_cache_dir()),
-        )
-    )

openml/config.py

@@ -540,10 +540,10 @@ def _sync_api_config() -> None:
 
     APIBackend.set_config_values(
         {
+            "cache_dir": cache_dir,
             "api_configs.v1.server": v1_server,
             "api_configs.v1.base_url": v1_base_url,
             "api_configs.v1.api_key": apikey,
-            "cache.dir": cache_dir,
             "connection.retry_policy": connection_retry_policy,
             "connection.retries": connection_n_retries,
         }