typechecking and Makefile for convenience

Author: mathematicalmichael

Makefile

@@ -0,0 +1,18 @@
+install:
+	bun install
+	uv sync --all-extras
+
+clean:
+	rm -rf node_modules .venv
+
+build:
+	bun run build
+	uv build
+
+test:
+	uv run pytest hiplot --durations=10
+
+check:
+	# uv run mypy --strict --implicit-reexport hiplot
+	uv run ty check
+

hiplot/__init__.py

@@ -5,7 +5,7 @@
 from .experiment import (Experiment, ExperimentFetcherDoesntApply, ExperimentValidationError, ExperimentValidationCircularRef,
                          ExperimentValidationMissingParent, Datapoint, ExperimentDisplayed, ValueDef, ValueType, Displays)
 from .server import run_server, run_server_main
-from .pkginfo import version as __version__, package_name
+from .pkginfo import __version__, package_name
 
 from . import fetchers
 

hiplot/pkginfo.py

@@ -7,16 +7,16 @@
 # Dynamic version from installed package metadata
 # This ensures __version__ matches pyproject.toml when installed via pip
 try:
-    from importlib.metadata import version, PackageNotFoundError
+    from importlib.metadata import version as get_version, PackageNotFoundError
     try:
-        version = version(package_name)
+        __version__ = get_version(package_name)
     except PackageNotFoundError:
         # Package is not installed (e.g., running from source checkout)
-        version = "0.0.0.dev0"
+        __version__ = "0.0.0.dev0"
 except ImportError:
     # Python < 3.8 fallback
     try:
         import importlib_metadata
-        version = importlib_metadata.version(package_name)
+        __version__ = importlib_metadata.version(package_name)
     except Exception:
-        version = "0.0.0.dev0"
+        __version__ = "0.0.0.dev0"

hiplot/render.py

@@ -36,7 +36,7 @@ def html_inlinize(html: str, replace_local: bool = True) -> str:
     static_root = str(Path(__file__).parent)
     soup = BeautifulSoup(html, "html.parser")
     for i in soup.find_all("link"):
-        href = i["href"]
+        href = str(i["href"])
         if href.startswith("http") or href.startswith("//"):
             continue
         if not replace_local:
@@ -54,7 +54,7 @@ def html_inlinize(html: str, replace_local: bool = True) -> str:
             i["href"] = f"data:{SUFFIX_TO_TYPE[file.suffix]};base64,{base64.b64encode(file.open('rb').read()).decode('ascii')}"
     for i in soup.find_all("script"):
         try:
-            src = i["src"]
+            src = str(i["src"])
         except KeyError:
             continue
         if src.startswith("http") or src.startswith("//"):

hiplot/server.py

@@ -51,7 +51,7 @@ def data() -> Any:  # pylint: disable=unused-variable
 
 def run_server_main() -> int:
     parser = argparse.ArgumentParser(prog="HiPlot", description="Start HiPlot webserver")
-    parser.add_argument('--version', action='version', version=f'{pkginfo.package_name} {pkginfo.version}')
+    parser.add_argument('--version', action='version', version=f'{pkginfo.package_name} {pkginfo.__version__}')
     parser.add_argument("--host", type=str, default="127.0.0.1")
     parser.add_argument("--port", type=int, default=5005)
     parser.add_argument("--dev", action='store_true', help="Enable Flask Debug mode (watches for files modifications, etc..)")

pyproject.toml

@@ -61,10 +61,10 @@ all = [
 # Development dependencies
 dev = [
     "pytest",
-    "mypy",
+    "ty",
     "ipykernel",
     "wheel",
-    "selenium",
+    # "selenium",
     "mistune==0.8.4",
     "twine",
     "pre-commit",
@@ -74,14 +74,14 @@ dev = [
     "optuna",
     "sphinx==5.2.0",
     "guzzle_sphinx_theme==0.7.11",
-    "m2r2==0.3.3",
+    "m2r2>=0.3.4",
     "flask",
     "flask-compress",
     "ipython>=7.0.1",
 ]
 
 [project.scripts]
-hiplot = "hiplot.server:run_server_main"
+hiplot-mm = "hiplot.server:run_server_main"
 hiplot-render = "hiplot.render:hiplot_render_main"
 
 [project.urls]