EliminateDiagonalSegments runs in the hybrid baseline finalization but
large diagonals can re-appear during iterative optimization. Added a
conditional elimination pass in the winner refinement when
LongDiagonalViolations > 0.
NodeSpacing=40 retained (default). Tested 42/45/50/60 — each creates
different violations because the routing is tuned for 40. Wider spacing
needs its own tuning pass.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The final ApplyFinalBoundarySlotPolish (39s) didn't reduce violations
(4->4) but ran unconditionally. Now skipped in low-wave path.
Layout-only speed: 2m05s (down from 2m46s with optimization, was 14s
before quality pipeline). Artifact test still passes (1m50s).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add per-edge node-crossing and shared-lane pre-check before expensive
ComputeScore. Skip final boundary-slot snap in low-wave path (no-op:
violations 4->4). Boundary-slot polish kept (fixes entry-angle).
Layout-only speed regressed from 14s to ~2m due to quality pipeline
additions (boundary-slot polish 49s, detour polish 25s, per-edge
gateway redirect+scoring). This is the tradeoff for zero-violation
artifact quality. Speed optimization is future work.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Two under-node fix strategies in the winner refinement:
1. Long sweeps (> 40% graph width): route through top corridor at
graphMinY - 56, with perpendicular exit stub. Fixes edge/20.
2. Medium sweeps near graph bottom: route through bottom corridor at
graphMaxY + 32 when the safe push-down Y would exceed graph bounds.
Fixes edge/25 (was 29px gap, now routes below blocking nodes).
Both under-node geometry violations eliminated. Edge/25 gains a
below-graph flag (Y=803 vs graphMaxY=771) which the FinalScore
adjustment handles as a corridor routing pattern.
Also adds target-join face reassignment infrastructure (redirects
outer edge to target's right face) — evaluates but not yet promoted
for the current fixture.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Detects horizontal segments > 40% of graph width with under-node
violations and reroutes them through the top corridor (Y = graphMinY
- 56), similar to backward edge routing. The corridor path includes a
24px perpendicular exit stub that survives NormalizeBoundaryAngles
without being collapsed.
Fixes edge/20 (3076px horizontal sweep from Load Configuration to End)
which previously crossed 10 layers at Y=201, passing under intermediate
nodes. Now routes above the graph at Y=-24.
Remaining geometry violations: 2 (target-join edge/32+33, under-node
edge/25).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Runs ElevateUnderNodeViolations as a final pass using weighted score
comparison (Score.Value) instead of per-category gating. Under-node
(100K penalty) is worth more than detour (50K), so trading one for
the other is a net score improvement.
Currently no change to the document fixture — the elevation logic's
internal guards find nothing new to elevate after the standard polish
stages. The remaining under-node edges (edge/20 3076px sweep, edge/25
29px gap) need corridor re-routing, not segment elevation.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The short-stub fallback in NormalizeExitPath fixes 2 entry-angle violations
(edge/7, edge/27) that persisted because the default long-stub normalization
created horizontal segments crossing nodes in occupied Y-bands. When the long
stub fails HasClearSourceExitSegment, the normalizer now tries a 24px short
stub that creates a perpendicular dog-leg exit avoiding the blocking node.
Also adds boundary-first routing infrastructure (not yet active in the main
path) including global boundary slot pre-computation, A* routing with
pre-assigned slots, coordinated cluster repair with net-total promotion
criterion, and gateway target approach overshoot clipping. The net-total
criterion (CountTotalHardViolations) is proven to reduce violations from
10 to 7 but requires expensive BuildFinalRestabilizedCandidate calls that
exceed the 15s speed budget.
Root cause analysis confirms the remaining 8 violations (3 gateway hooks,
1 target join, 1 shared lane, 3 under-node) are caused by Sugiyama node
placement creating routing corridors too narrow for clean edge routing.
The fix must happen upstream in node placement, not edge post-processing.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
