Skip to content

[codex] Add same-net trace consolidation phase#304

Open
dclawd88 wants to merge 1 commit into
tscircuit:mainfrom
dclawd88:codex/same-net-trace-consolidation
Open

[codex] Add same-net trace consolidation phase#304
dclawd88 wants to merge 1 commit into
tscircuit:mainfrom
dclawd88:codex/same-net-trace-consolidation

Conversation

@dclawd88
Copy link
Copy Markdown

@dclawd88 dclawd88 commented May 11, 2026

Summary

  • add a SameNetTraceConsolidationSolver phase after trace cleanup
  • snap close overlapping same-net horizontal/vertical trace segments together
  • reject snaps that would create different-net intersections
  • feed consolidated traces into the later label placement/example28 phases

/claim #29
/claim #34

Validation

  • npx --yes bun test tests/solvers/SameNetTraceConsolidationSolver/SameNetTraceConsolidationSolver.test.ts
  • npx --yes bun test tests/examples/example29.test.ts
  • npx --yes bun test tests/examples/example30.test.ts
  • npx --yes bun test tests/examples/example34.test.ts tests/solvers/SameNetTraceConsolidationSolver/SameNetTraceConsolidationSolver.test.ts
  • npx --yes bun run build
  • npx --yes bun run format:check
  • git diff --check
  • npx --yes bun test

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 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 11, 2026 7:44am

Request Review

@dclawd88
Copy link
Copy Markdown
Author

Status note: all required checks are green on this PR, including test, format-check, type-check, and Vercel. Ready for maintainer 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.

1 participant