Skip to content

fix: remove extra trace lines in post-processing step (issue #78)#201

Open
Wong789 wants to merge 6 commits intotscircuit:mainfrom
Wong789:fix-issue-78-wong789
Open

fix: remove extra trace lines in post-processing step (issue #78)#201
Wong789 wants to merge 6 commits intotscircuit:mainfrom
Wong789:fix-issue-78-wong789

Conversation

@Wong789
Copy link
Copy Markdown

@Wong789 Wong789 commented Apr 20, 2026

Fix for Issue #78: Remove extra trace lines in post-processing step

Problem

When multiple MSP pairs in the same net share a pin, they get routed independently and produce overlapping segments near the shared pin. Additionally, when a route is spliced into an existing path, consecutive duplicate points can appear, rendering as spurious extra trace lines.

Solution

  1. removeDuplicateConsecutivePoints - Removes consecutive duplicate points from trace paths. When a route is spliced, the concatenation can produce consecutive duplicate points (same x,y) which render as spurious extra trace lines.

  2. removeNetSegmentDuplicates - Removes duplicate trace segments within the same net. When multiple traces in the same net share pins, trims duplicate segments from the start and end of later traces while preserving path connectivity.

Files Changed

  • lib/solvers/TraceCleanupSolver/simplifyPath.ts - Added removeDuplicateConsecutivePoints
  • lib/solvers/TraceCleanupSolver/sub-solver/UntangleTraceSubsolver.ts - Apply deduplication when splicing routes
  • lib/solvers/TraceCleanupSolver/removeNetSegmentDuplicates.ts - New file for cross-trace deduplication
  • lib/solvers/TraceCleanupSolver/TraceCleanupSolver.ts - Apply removeNetSegmentDuplicates at output

/claim #78

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 20, 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 Apr 20, 2026 3:24pm

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