Skip to content

feat: merge close same-net trace segments#322

Open
Havarry wants to merge 15 commits into
tscircuit:mainfrom
Havarry:main
Open

feat: merge close same-net trace segments#322
Havarry wants to merge 15 commits into
tscircuit:mainfrom
Havarry:main

Conversation

@Havarry
Copy link
Copy Markdown

@Havarry Havarry commented May 12, 2026

Summary

  • add a TraceSegmentMergingSolver phase after long-distance trace generation
  • merge trace segments on the same global net when their closest endpoints are within a small threshold
  • feed merged traces into overlap shifting and later pipeline phases
  • add focused tests for same-net merge behavior and different-net non-merge behavior

Verification

  • /Users/wz/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin/node ./node_modules/typescript/bin/tsc --noEmit
  • /Users/wz/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin/node ./node_modules/tsup/dist/cli-node.js lib/index.ts --format esm --dts
  • npx bun test tests/solvers/TraceSegmentMergingSolver/trace-segment-merging-solver.test.ts tests/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver_repro01.test.ts

/claim #29

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 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 12, 2026 4:03pm

Request Review

Havarry added 2 commits May 12, 2026 23:55
Updated various circle and polyline elements in the SVG snapshot to reflect new coordinates and labels.
Updated circle and polyline elements in SVG snapshot.
Removed unnecessary data-label attributes from circles in SVG snapshot.
@Havarry
Copy link
Copy Markdown
Author

Havarry commented May 13, 2026

Superseded by clean resubmission #327. Please review #327 for the final target diff for #29.

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