adjusting to new changes

openml · satvshr · Feb 3, 2026 · Feb 3, 2026 · Feb 3, 2026 · Feb 3, 2026
commit 021c1ed60b6fc11f5bf0b229331409f82b65e1ed
diff --git a/openml/_api/clients/http.py b/openml/_api/clients/http.py
@@ -1,13 +1,12 @@
 from __future__ import annotations
 
-import hashlib
 import json
 import logging
 import math
 import random
 import time
 import xml
-from collections.abc import Mapping, Callable
+from collections.abc import Mapping
 from pathlib import Path
 from typing import Any
 from urllib.parse import urlencode, urljoin, urlparse
@@ -325,7 +324,6 @@ def request(
         use_cache: bool = False,
         reset_cache: bool = False,
         use_api_key: bool = False,
-        md5_checksum: str | None,
         **request_kwargs: Any,
     ) -> Response:
         url = urljoin(self.server, urljoin(self.base_url, path))
@@ -381,29 +379,19 @@ def request(
 
         assert response is not None
 
-        if md5_checksum is not None:
-            self._verify_checksum(response, md5_checksum)
-
         if use_cache and self.cache is not None:
             cache_key = self.cache.get_key(url, params)
             self.cache.save(cache_key, response)
 
         return response
 
-    def _verify_checksum(self, response: Response, md5_checksum: str) -> None:
-        # ruff sees hashlib.md5 as insecure
-        actual = hashlib.md5(response.content).hexdigest()  # noqa: S324
-        if actual != md5_checksum:
-            raise ValueError(f"MD5 checksum mismatch: expected {md5_checksum}, got {actual}")
-
     def get(
         self,
         path: str,
         *,
         use_cache: bool = False,
         reset_cache: bool = False,
         use_api_key: bool = False,
-        md5_checksum: str | None = None,
         **request_kwargs: Any,
     ) -> Response:
         return self.request(
@@ -412,7 +400,6 @@ def get(
             use_cache=use_cache,
             reset_cache=reset_cache,
             use_api_key=use_api_key,
-            md5_checksum=md5_checksum,
             **request_kwargs,
         )
 
@@ -440,31 +427,4 @@ def delete(
             use_cache=False,
             use_api_key=True,
             **request_kwargs,
-        )
-
-    def download(
-        self,
-        url: str,
-        handler: Callable[[Response, Path, str], Path] | None = None,
-        encoding: str = "utf-8",
-        file_name: str = "response.txt",
-        md5_checksum: str | None = None,
-    ) -> Path:
-        # TODO(Shrivaths) find better way to get base path
-        base = self.cache.path if self.cache is not None else Path("~/.openml/cache")
-        file_path = base / "downloads" / urlparse(url).path.lstrip("/") / file_name
-        file_path = file_path.expanduser()
-        file_path.parent.mkdir(parents=True, exist_ok=True)
-        if file_path.exists():
-            return file_path
-
-        response = self.get(url, md5_checksum=md5_checksum)
-        if handler is not None:
-            return handler(response, file_path, encoding)
-
-        return self._text_handler(response, file_path, encoding)
-
-    def _text_handler(self, response: Response, path: Path, encoding: str) -> Path:
-        with path.open("w", encoding=encoding) as f:
-            f.write(response.text)
-        return path
+        )
diff --git a/openml/_api/resources/base.py b/openml/_api/resources/base.py