Skip to content

Combine nearby same-net trace segments#372

Open
thebasedcapital wants to merge 4 commits into
tscircuit:mainfrom
thebasedcapital:fix-same-net-trace-combiner
Open

Combine nearby same-net trace segments#372
thebasedcapital wants to merge 4 commits into
tscircuit:mainfrom
thebasedcapital:fix-same-net-trace-combiner

Conversation

@thebasedcapital
Copy link
Copy Markdown

@thebasedcapital thebasedcapital commented May 20, 2026

Summary

  • add a SameNetTraceCombinerSolver pipeline phase after trace cleanup and before relabeling
  • snap close, parallel, internal same-net trace segments onto a shared route
  • preserve boundary segments so pin anchors are not moved
  • skip candidate snaps that would introduce chip collisions or cross other-net traces
  • add focused coverage for same-net combining, different-net safety, collision safety, and pin anchoring
  • update visual snapshots for the examples whose same-net routes are intentionally consolidated

Fixes #29
/claim #29

Tests

  • bunx tsc --noEmit
  • bun run build
  • bun test tests/solvers/SameNetTraceCombinerSolver.test.ts with bunfig preload temporarily disabled because install used --ignore-scripts for safety, so sharp's native postinstall artifact was not present
  • bunx biome format
  • git diff --check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
schematic-trace-solver Ready Ready Preview, Comment May 20, 2026 4:12am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New Phase To combine same-net trace segments that are close together

1 participant