metrics: tighten Metrics page copy

Fix the "Editing tasks" anchor landing short (bottom spacer), drop the
per-section "Full derivation" links in favor of a single docs/metrics.md
pointer at the end, trim wording (Boolean kernel, "(pieces)", "(e.g.
through-holes)", "not the mean", the report-overlay note), and remove
em-dashes.

Co-authored-by: Cursor <cursoragent@cursor.com>

metrics_page.py

@@ -98,7 +98,8 @@ code { background: #eef0f4; padding: 1px 5px; border-radius: 4px;
 table { border-collapse: collapse; width: 100%; margin: 12px 0; font-size: 0.92em; }
 th, td { border: 1px solid #e3e5ea; padding: 7px 10px; text-align: left; }
 th { background: #f5f7fa; }
-.deep { font-size: 0.86em; color: #5b6170; margin-top: 12px; }
+.deep { font-size: 0.9em; color: #5b6170; margin: 20px 0 0; }
+.endspace { height: 60vh; }
 .toc { background: #fff; border: 1px solid #e3e5ea; border-radius: 12px;
        padding: 14px 20px; margin: 16px 0; }
 .toc ul { margin: 6px 0 0; padding-left: 18px; }
@@ -115,18 +116,11 @@ figure.fig figcaption { font-size: 0.84em; color: #5b6170; margin-top: 6px;
 
 def _section(
     *, anchor: str, css_class: str, tag: str, title: str, body: str,
-    deep_dive: str | None = None,
 ) -> str:
-    deep = (
-        f'<p class="deep">Full derivation: '
-        f'<a href="{deep_dive}" target="_blank" rel="noopener">{deep_dive}</a></p>'
-        if deep_dive
-        else ""
-    )
     return (
         f'<section class="card {css_class}" id="{anchor}">'
         f'<h2><span class="axis-tag">{tag}</span>{title}</h2>'
-        f"{body}{deep}"
+        f"{body}"
         "</section>"
     )
 
@@ -150,8 +144,8 @@ def build_metrics_page() -> str:
             "          = 0.4*shape + 0.4*interface + 0.2*topology          otherwise"
             "</pre>"
             "<p class='note'>(This is the <b>generation</b> composition. "
-            "<b>Editing</b> tasks renormalize the shape axis and reweight — "
-            f'see <a href="#{a["editing"]}">Editing tasks</a>.)</p>'
+            "<b>Editing</b> tasks renormalize the shape axis and reweight; "
+            f'see <a href="#{a["editing"]}">Editing tasks</a> below.)</p>'
             "<table><thead><tr><th>Component</th><th>Range</th>"
             "<th>What it asks</th></tr></thead><tbody>"
             f'<tr><td><a href="#{a["validity"]}">CAD Validity</a> (gate)</td>'
@@ -164,7 +158,7 @@ def build_metrics_page() -> str:
             "<td>[0, 1]</td><td>Does it bolt up to the same fixture?</td></tr>"
             "</tbody></table>"
             "<h3>Why three axes</h3>"
-            "<p>They are orthogonal by construction — each catches errors the "
+            "<p>They are orthogonal by construction: each catches errors the "
             "others are blind to:</p>"
             "<ul>"
             "<li><b>Shape</b> catches wrong bulk geometry; blind to topology.</li>"
@@ -176,7 +170,6 @@ def build_metrics_page() -> str:
             "<p class='note'>Outputs are rigidly aligned to the ground truth "
             "(rotation + translation only, never scale) before scoring.</p>"
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics.md",
     )
 
     validity = _section(
@@ -190,16 +183,15 @@ def build_metrics_page() -> str:
             "<code>cad_score = 0</code>, so an invalid solid never beats a worse "
             "but valid one. Passing requires all of:</p>"
             "<ol>"
-            "<li><b>Well-formed BREP</b> — no per-face / edge / vertex errors "
+            "<li><b>Well-formed BREP</b>: no per-face / edge / vertex errors "
             "(self-intersecting wires, edges off their surface, etc.).</li>"
-            "<li><b>Watertight</b> — every shell is closed; no naked or free "
+            "<li><b>Watertight</b>: every shell is closed; no naked or free "
             "edges.</li>"
-            "<li><b>Meshable as a closed orientable manifold</b> — tessellates "
+            "<li><b>Meshable as a closed orientable manifold</b>: tessellates "
             "to a manifold, closed (3F = 2E), orientation-consistent triangle "
             "mesh.</li>"
             "</ol>"
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics/cad_validity.md",
     )
 
     shape = _section(
@@ -222,12 +214,11 @@ def build_metrics_page() -> str:
             "combine into F1.</p>"
             "<h3>Volume IoU</h3>"
             "<p>Shared volume of the two solids over their combined volume "
-            "(intersection over union), via a Boolean kernel.</p>"
+            "(intersection over union).</p>"
             "<p class='note'>Both use a tolerance proportional to part size, so "
-            "small features can move without shifting the score — those are "
+            "small features can move without shifting the score; those are "
             f'covered by <a href="#{a["interface"]}">interface match</a>.</p>'
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics/shape_similarity.md",
     )
 
     topology = _section(
@@ -240,9 +231,8 @@ def build_metrics_page() -> str:
             "through-holes, and internal voids? It compares the three "
             "<b>Betti numbers</b> of the solid:</p>"
             "<ul>"
-            "<li><b>b&#8320;</b>: connected solid components (pieces).</li>"
-            "<li><b>b&#8321;</b>: independent through-handles (e.g. "
-            "through-holes).</li>"
+            "<li><b>b&#8320;</b>: connected solid components.</li>"
+            "<li><b>b&#8321;</b>: independent through-handles.</li>"
             "<li><b>b&#8322;</b>: enclosed internal voids (cavities).</li>"
             "</ul>"
             "<p>Each axis gets a fuzzy log-ratio against GT, sharpened by "
@@ -251,14 +241,13 @@ def build_metrics_page() -> str:
             "s_i = ((min(cand,gt) + 1) / (max(cand,gt) + 1)) ^ 2\n"
             "topology_match = s_0 * s_1 * s_2"
             "</pre>"
-            "<p>The product (not the mean) means one wrong count collapses the "
+            "<p>The product means one wrong count collapses the "
             "score: topology is discrete, so two of three right is not a partial "
             "match. Example: GT (1,2,0) vs candidate (1,4,0) scores "
             "(3/5)&#178; = 0.36. Blind features (blind pockets, fillets, "
             "chamfers) are topologically trivial and covered by the other "
             "axes.</p>"
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics/topo_match.md",
     )
 
     interface = _section(
@@ -271,13 +260,13 @@ def build_metrics_page() -> str:
             "region of space the candidate must match in shape, size, and "
             "position:</p>"
             "<ul>"
-            "<li><b>Keep-out (KOR)</b> — must be empty (a bolt hole, a slot).</li>"
-            "<li><b>Keep-in (KIR)</b> — must be solid (a locating boss, a "
+            "<li><b>Keep-out (KOR)</b>: must be empty (a bolt hole, a slot).</li>"
+            "<li><b>Keep-in (KIR)</b>: must be solid (a locating boss, a "
             "pin).</li>"
             "</ul>"
             "<h3>Mating groups</h3>"
             "<p>The features that must seat together against a single fixture "
-            "form one <b>mating group</b> — here, two bolt holes and a slot that "
+            "form one <b>mating group</b>: here, two bolt holes and a slot that "
             "one jig drops into. A part can have several independent groups (say "
             "a bolt pattern on one face and a boss on another), and each group "
             "is scored on its own.</p>"
@@ -292,24 +281,19 @@ def build_metrics_page() -> str:
             "<h3>Scoring</h3>"
             "<p>Per group:</p>"
             "<ol>"
-            "<li><b>Per-feature fit</b> — volumetric IoU against the region "
+            "<li><b>Per-feature fit</b>: volumetric IoU against the region "
             "(with a thin shell of opposite material, so both oversize and "
             "undersize lose points).</li>"
-            "<li><b>Bounded pose search</b> — &#177;1&#176; and &#177;1% of part "
+            "<li><b>Bounded pose search</b>: &#177;1&#176; and &#177;1% of part "
             "size per axis, so a feature isn't penalized for the residual of "
             "whole-part alignment.</li>"
-            "<li><b>Pass/fail ramp</b> — IoU &#8805; 0.95 &#8594; 1, &#8804; 0.80 "
+            "<li><b>Pass/fail ramp</b>: IoU &#8805; 0.95 &#8594; 1, &#8804; 0.80 "
             "&#8594; 0, linear between; a sloppy fit scores 0.</li>"
             "</ol>"
             "<p>A group scores as its <b>worst</b> feature (the minimum); the "
             "fixture scores as the <b>mean</b> over its groups, so nailing one "
             "interface and missing another still earns partial credit.</p>"
-            "<p class='note'>In the report's overlay: <b>blue</b> where it fits, "
-            "<b>red</b> where the candidate has material it shouldn't (too much), "
-            "<b>amber</b> where it's missing material it should have (too "
-            "little).</p>"
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics/interface_match.md",
     )
 
     editing = _section(
@@ -334,12 +318,11 @@ def build_metrics_page() -> str:
             "therefore caps at 0.3 + 0.1 = 0.4, and any real shape improvement "
             "clears it.</p>"
         ),
-        deep_dive=f"{_DOCS_BASE}/metrics.md#editing-tasks-no-op-renormalization",
     )
 
     toc = (
         '<nav class="toc"><b>On this page</b><ul>'
-        f'<li><a href="#{a["cad_score"]}">CAD Score &mdash; how one part is scored</a></li>'
+        f'<li><a href="#{a["cad_score"]}">CAD Score: how one part is scored</a></li>'
         f'<li><a href="#{a["validity"]}">CAD Validity (gate)</a></li>'
         f'<li><a href="#{a["shape"]}">Shape Similarity</a></li>'
         f'<li><a href="#{a["topology"]}">Topology Match</a></li>'
@@ -348,19 +331,29 @@ def build_metrics_page() -> str:
         "</ul></nav>"
     )
 
+    footer = (
+        '<p class="deep">For the full definitions and derivations, see the '
+        f'metrics reference in the code: '
+        f'<a href="{_DOCS_BASE}/metrics.md" target="_blank" rel="noopener">'
+        "docs/metrics.md</a>.</p>"
+        # Trailing space so the last section can scroll to the top of the
+        # viewport when reached via an in-page anchor (e.g. the "Editing
+        # tasks" link); without it a near-bottom target lands mid-screen.
+        '<div class="endspace" aria-hidden="true"></div>'
+    )
+
     return (
         "<!DOCTYPE html><html lang='en'><head>"
         "<meta charset='utf-8'>"
         "<meta name='viewport' content='width=device-width, initial-scale=1'>"
-        "<title>CADGenBench &mdash; Metrics</title>"
+        "<title>CADGenBench Metrics</title>"
         f"<style>{_CSS}</style>"
         "</head><body>"
         "<h1>Metrics</h1>"
         "<p class='lede'>How CADGenBench scores one generated CAD part against "
         "the ground truth. These metrics are new, so this page explains each "
-        "one; the canonical reference lives in the "
-        f'<a href="{_DOCS_BASE}/metrics.md" target="_blank" rel="noopener">'
-        "code repo</a>.</p>"
+        "one.</p>"
         f"{toc}{overview}{validity}{shape}{topology}{interface}{editing}"
+        f"{footer}"
         "</body></html>"
     )