Skip to content

Add same-net trace combining phase#365

Open
WhiteEagle-12 wants to merge 1 commit into
tscircuit:mainfrom
WhiteEagle-12:codex/same-net-trace-combining
Open

Add same-net trace combining phase#365
WhiteEagle-12 wants to merge 1 commit into
tscircuit:mainfrom
WhiteEagle-12:codex/same-net-trace-combining

Conversation

@WhiteEagle-12
Copy link
Copy Markdown

Summary

Adds a focused post-cleanup SameNetTraceCombiningSolver phase for #29. The new phase:

  • groups routed traces by globalConnNetId
  • detects close overlapping internal horizontal/vertical segments on the same net
  • aligns the movable segment onto the longer segment's axis
  • preserves endpoint/pin segments
  • avoids changes that increase strict crossings with different-net traces

The phase is wired after TraceCleanupSolver, before the second net-label placement pass and Example28Solver, so downstream placement uses the combined trace geometry.

Verification

  • npx --yes bun test tests/solvers/SameNetTraceCombiningSolver/SameNetTraceCombiningSolver.test.ts
  • npx --yes bun test tests/examples/example29.test.ts tests/examples/example30.test.ts tests/examples/example34.test.ts
  • npx --yes -p typescript@5.9.3 tsc --noEmit
  • npm run build

/claim #29

@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 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 17, 2026 7:30pm

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.

1 participant