Strip ties and beams for score preview fallback

apps/theme_lab/app.py

@@ -46,6 +46,7 @@ MARKOV_CACHE_DIR = ROOT / "models" / "theme_lab_markov_cache"
 DEFAULT_MARKOV_CACHE = MARKOV_CACHE_DIR / "default.pkl"
 MARKOV_CACHE_FORMAT_VERSION = 1
 MARKOV_PRECOMPUTED_WARM_SAMPLES = 4
+SCORE_PREVIEW_RENDER_VERSION = "strip-ties-beams-v1"
 GENERATED_ROOT.mkdir(parents=True, exist_ok=True)
 CATALOG_SCORE_ROOT.mkdir(parents=True, exist_ok=True)
 
@@ -126,6 +127,7 @@ def health() -> dict[str, object]:
         "verovio": verovio_cli or verovio_python,
         "verovio_cli": verovio_cli,
         "verovio_python": verovio_python,
+        "score_preview_render_version": SCORE_PREVIEW_RENDER_VERSION,
         "markov_cache_entries": len(_markov_cache),
         "default_markov_cache": DEFAULT_MARKOV_CACHE.exists(),
         "transformer_checkpoint": DEFAULT_TRANSFORMER_CHECKPOINT.exists(),
@@ -383,32 +385,44 @@ def has_python_verovio_renderer() -> bool:
     return True
 
 
-def musicxml_without_ties(musicxml_data: str) -> str | None:
+def musicxml_preview_data(musicxml_data: str, *, strip_ties: bool = False, strip_beams: bool = False) -> str | None:
     try:
         root = ET.fromstring(musicxml_data)
     except ET.ParseError:
         return None
     for note in root.findall(".//note"):
-        for tie in list(note.findall("tie")):
-            note.remove(tie)
+        if strip_ties:
+            for tie in list(note.findall("tie")):
+                note.remove(tie)
         notations = note.find("notations")
-        if notations is None:
-            continue
-        for tied in list(notations.findall("tied")):
-            notations.remove(tied)
-        if len(notations) == 0:
-            note.remove(notations)
+        if strip_ties and notations is not None:
+            for tied in list(notations.findall("tied")):
+                notations.remove(tied)
+            if len(notations) == 0:
+                note.remove(notations)
+        if strip_beams:
+            for beam in list(note.findall("beam")):
+                note.remove(beam)
     ET.indent(root)
     return ET.tostring(root, encoding="unicode", short_empty_elements=True)
 
 
-def render_musicxml_with_python_verovio(musicxml_path: Path, svg_path: Path, *, strip_ties: bool = False) -> Path | None:
+def render_musicxml_with_python_verovio(
+    musicxml_path: Path,
+    svg_path: Path,
+    *,
+    strip_ties: bool = False,
+    strip_beams: bool = False,
+) -> Path | None:
     try:
         import verovio
 
         musicxml_data = musicxml_path.read_text(encoding="utf-8")
-        if strip_ties:
-            musicxml_data = musicxml_without_ties(musicxml_data) or musicxml_data
+        if strip_ties or strip_beams:
+            musicxml_data = (
+                musicxml_preview_data(musicxml_data, strip_ties=strip_ties, strip_beams=strip_beams)
+                or musicxml_data
+            )
         toolkit = verovio.toolkit()
         toolkit.setOptions({"scale": 42})
         svg = toolkit.renderData(musicxml_data, {}) if hasattr(toolkit, "renderData") else ""
@@ -431,6 +445,9 @@ def render_musicxml_to_svg(musicxml_path: Path) -> Path | None:
     if python_svg is not None:
         return python_svg
     python_svg = render_musicxml_with_python_verovio(musicxml_path, svg_path, strip_ties=True)
+    if python_svg is not None:
+        return python_svg
+    python_svg = render_musicxml_with_python_verovio(musicxml_path, svg_path, strip_ties=True, strip_beams=True)
     if python_svg is not None:
         return python_svg
     verovio = shutil.which("verovio")