rename config: timeout -> timeout_seconds

Author: geetu040

openml/_api/clients/http.py

@@ -116,15 +116,15 @@ def __init__(  # noqa: PLR0913
         server: str,
         base_url: str,
         api_key: str,
-        timeout: int,
+        timeout_seconds: int,
         retries: int,
         retry_policy: RetryPolicy,
         cache: HTTPCache | None = None,
     ) -> None:
         self.server = server
         self.base_url = base_url
         self.api_key = api_key
-        self.timeout = timeout
+        self.timeout_seconds = timeout_seconds
         self.retries = retries
         self.retry_policy = retry_policy
         self.cache = cache
@@ -343,7 +343,7 @@ def request(
         headers = request_kwargs.pop("headers", {}).copy()
         headers.update(self.headers)
 
-        timeout = request_kwargs.pop("timeout", self.timeout)
+        timeout = request_kwargs.pop("timeout", self.timeout_seconds)
         files = request_kwargs.pop("files", None)
 
         if use_cache and not reset_cache and self.cache is not None:

openml/_api/setup/builder.py

@@ -33,7 +33,7 @@ def build(cls, config: Config) -> APIBackendBuilder:
             server=primary_api_config.server,
             base_url=primary_api_config.base_url,
             api_key=primary_api_config.api_key,
-            timeout=config.connection.timeout,
+            timeout_seconds=config.connection.timeout_seconds,
             retries=config.connection.retries,
             retry_policy=config.connection.retry_policy,
             cache=http_cache,
@@ -51,7 +51,7 @@ def build(cls, config: Config) -> APIBackendBuilder:
             server=fallback_api_config.server,
             base_url=fallback_api_config.base_url,
             api_key=fallback_api_config.api_key,
-            timeout=config.connection.timeout,
+            timeout_seconds=config.connection.timeout_seconds,
             retries=config.connection.retries,
             retry_policy=config.connection.retry_policy,
             cache=http_cache,

openml/_api/setup/config.py

@@ -18,7 +18,7 @@ class APIConfig:
 class ConnectionConfig:
     retries: int
     retry_policy: RetryPolicy
-    timeout: int
+    timeout_seconds: int
 
 
 @dataclass
@@ -51,7 +51,7 @@ class Config:
         default_factory=lambda: ConnectionConfig(
             retries=5,
             retry_policy=RetryPolicy.HUMAN,
-            timeout=10,
+            timeout_seconds=10,
         )
     )
 

openml/testing.py

@@ -286,7 +286,7 @@ class TestAPIBase(unittest.TestCase):
     server: str
     base_url: str
     api_key: str
-    timeout: int
+    timeout_seconds: int
     retries: int
     retry_policy: RetryPolicy
     dir: str
@@ -298,7 +298,7 @@ def setUp(self) -> None:
         self.server = "https://test.openml.org/"
         self.base_url = "api/v1/xml"
         self.api_key = "normaluser"
-        self.timeout = 10
+        self.timeout_seconds = 10
         self.retries = 3
         self.retry_policy = RetryPolicy.HUMAN
         self.dir = "test_cache"
@@ -312,7 +312,7 @@ def setUp(self) -> None:
             server=self.server,
             base_url=self.base_url,
             api_key=self.api_key,
-            timeout=self.timeout,
+            timeout_seconds=self.timeout_seconds,
             retries=self.retries,
             retry_policy=self.retry_policy,
             cache=self.cache,
@@ -340,7 +340,7 @@ def _get_http_client(  # noqa: PLR0913
         server: str,
         base_url: str,
         api_key: str,
-        timeout: int,
+        timeout_seconds: int,
         retries: int,
         retry_policy: RetryPolicy,
         cache: HTTPCache | None = None,
@@ -349,7 +349,7 @@ def _get_http_client(  # noqa: PLR0913
             server=server,
             base_url=base_url,
             api_key=api_key,
-            timeout=timeout,
+            timeout_seconds=timeout_seconds,
             retries=retries,
             retry_policy=retry_policy,
             cache=cache,