Prefer Python Verovio for score rendering

apps/theme_lab/app.py

@@ -116,10 +116,14 @@ def index() -> FileResponse:
 
 @app.get("/api/health")
 def health() -> dict[str, object]:
+    verovio_cli = shutil.which("verovio") is not None
+    verovio_python = has_python_verovio_renderer()
     return {
         "ok": True,
         "database": DB_PATH.exists(),
-        "verovio": has_verovio_renderer(),
+        "verovio": verovio_cli or verovio_python,
+        "verovio_cli": verovio_cli,
+        "verovio_python": verovio_python,
         "markov_cache_entries": len(_markov_cache),
         "default_markov_cache": DEFAULT_MARKOV_CACHE.exists(),
         "transformer_checkpoint": DEFAULT_TRANSFORMER_CHECKPOINT.exists(),
@@ -366,6 +370,10 @@ def sequence_note_events(sequence: tuple[Symbol, ...], key_name: str) -> list[di
 def has_verovio_renderer() -> bool:
     if shutil.which("verovio") is not None:
         return True
+    return has_python_verovio_renderer()
+
+
+def has_python_verovio_renderer() -> bool:
     try:
         import verovio  # noqa: F401
     except ImportError:
@@ -379,7 +387,10 @@ def render_musicxml_with_python_verovio(musicxml_path: Path, svg_path: Path) ->
 
         toolkit = verovio.toolkit()
         toolkit.setOptions({"scale": 42})
-        if not toolkit.loadFile(str(musicxml_path)) and not toolkit.loadData(musicxml_path.read_text(encoding="utf-8")):
+        loaded = toolkit.loadFile(str(musicxml_path))
+        if not loaded and hasattr(toolkit, "loadData"):
+            loaded = toolkit.loadData(musicxml_path.read_text(encoding="utf-8"))
+        if not loaded:
             return None
         svg_path.write_text(toolkit.renderToSVG(1), encoding="utf-8")
     except Exception:
@@ -389,9 +400,12 @@ def render_musicxml_with_python_verovio(musicxml_path: Path, svg_path: Path) ->
 
 def render_musicxml_to_svg(musicxml_path: Path) -> Path | None:
     svg_path = musicxml_path.with_suffix(".svg")
+    python_svg = render_musicxml_with_python_verovio(musicxml_path, svg_path)
+    if python_svg is not None:
+        return python_svg
     verovio = shutil.which("verovio")
     if verovio is None:
-        return render_musicxml_with_python_verovio(musicxml_path, svg_path)
+        return None
     try:
         subprocess.run(
             [verovio, "-s", "42", "-o", str(svg_path), str(musicxml_path)],
@@ -403,8 +417,8 @@ def render_musicxml_to_svg(musicxml_path: Path) -> Path | None:
             timeout=20,
         )
     except (subprocess.SubprocessError, OSError):
-        return render_musicxml_with_python_verovio(musicxml_path, svg_path)
-    return svg_path if svg_path.exists() else render_musicxml_with_python_verovio(musicxml_path, svg_path)
+        return None
+    return svg_path if svg_path.exists() else None
 
 
 def markov_cache_key(request: GenerateRequest) -> tuple[object, ...]: