Skip to content

feat: connect close same-net trace segments#328

Open
autochamchikim-pixel wants to merge 1 commit into
tscircuit:mainfrom
autochamchikim-pixel:fix/combine-same-net-trace-segments
Open

feat: connect close same-net trace segments#328
autochamchikim-pixel wants to merge 1 commit into
tscircuit:mainfrom
autochamchikim-pixel:fix/combine-same-net-trace-segments

Conversation

@autochamchikim-pixel
Copy link
Copy Markdown

@autochamchikim-pixel autochamchikim-pixel commented May 13, 2026

Summary

  • Adds a dedicated SameNetTraceConnectorSolver pipeline phase after trace cleanup.
  • Detects close endpoints from separate trace segments that share the same globalConnNetId.
  • Adds short orthogonal connector traces to join those close same-net segments without touching different nets.
  • Feeds the connected trace output into later label-placement/example phases.

Test plan

  • npx -p typescript@5.6.3 tsc --noEmit
  • npx bun test tests/functions/sameNetTraceConnectorSolver.test.ts

Closes #29

Maintainer review notes

  • The solver phase is isolated after trace cleanup to minimize regression risk.
  • It only connects close endpoints sharing the same globalConnNetId; different nets are intentionally ignored.
  • Targeted tests cover same-net connection behavior and guard against cross-net joins.

@autochamchikim-pixel
Copy link
Copy Markdown
Author

/claim #29 #328

@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 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 13, 2026 5:23am

Request Review

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

Labels

None yet

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