Skip to content

Add same-net trace merging pipeline phase#309

Open
darshan-Jahagirdar wants to merge 1 commit into
tscircuit:mainfrom
darshan-Jahagirdar:codex/same-net-trace-merging
Open

Add same-net trace merging pipeline phase#309
darshan-Jahagirdar wants to merge 1 commit into
tscircuit:mainfrom
darshan-Jahagirdar:codex/same-net-trace-merging

Conversation

@darshan-Jahagirdar
Copy link
Copy Markdown

@darshan-Jahagirdar darshan-Jahagirdar commented May 11, 2026

Fixes #29
Fixes #34

/claim #29
/claim #34

Summary

  • add a SameNetTraceMergingSolver phase after trace cleanup
  • snap close, overlapping, parallel trace segments on the same global net onto a shared run while leaving different-net segments unchanged
  • feed merged traces into the final net-label placement and downstream cleanup phases
  • export the solver and add targeted unit coverage

Verification

  • npx --yes bun test tests/solvers/SameNetTraceMergingSolver/SameNetTraceMergingSolver.test.ts
  • npx --yes bun test tests/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver_repro03.test.ts
  • npx --yes bun test
  • npx --yes -p typescript@^5 tsc --noEmit
  • npm run build

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@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 8:16pm

Request Review

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

Projects

None yet

1 participant