Skip to content

feat: add SameNetTraceCombiningSolver pipeline phase (#29)#302

Closed
Mira-Mjodheim wants to merge 2 commits into
tscircuit:mainfrom
Mira-Mjodheim:feat/same-net-trace-combining-solver-restored
Closed

feat: add SameNetTraceCombiningSolver pipeline phase (#29)#302
Mira-Mjodheim wants to merge 2 commits into
tscircuit:mainfrom
Mira-Mjodheim:feat/same-net-trace-combining-solver-restored

Conversation

@Mira-Mjodheim
Copy link
Copy Markdown

This PR restores and continues the previous attempt #284, which was accidentally closed when the head repository was deleted during fork cleanup.

Implements SameNetTraceCombiningSolver for issue #29.

Scope:

  • combines adjacent same-net horizontal trace segments;
  • combines adjacent same-net vertical trace segments;
  • avoids combining segments from different nets;
  • avoids combining segments separated by large gaps;
  • includes tests covering same-net merge and non-merge cases.

Validation:

  • SameNetTraceCombiningSolver targeted tests pass.
  • Full test suite passes after updating expected snapshots for the intentional trace-combining output changes.

Notes:

@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 6:35am

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.

2 participants