Skip zero-length trace segments from duplicate route points#976
Skip zero-length trace segments from duplicate route points#976seveibar merged 2 commits intotscircuit:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🏃 Benchmark This PRRun benchmarks by commenting on this PR: Everything after Examples:
Use Any PR whose title contains |
|
@mohan-bee what motivates this fix? |
|
it seems fine but i'm curious how you found it, why it is a problem etc. |
@seveibar tscircuit/circuit-json-to-kicad#242 first i thought that is from kicad and then @imrishabh18 caught the core issue ! |
|
/benchmark |
🏃 Autorouting Benchmark Results📊 Main Branch ResultsDataset: dataset01
Details
📊 PR ResultsDataset: dataset01
Details
🔗 Workflow: View run |
|
Thank you for your contribution! 🎉 PR Rating: ⭐⭐⭐ Track your contributions and see the leaderboard at: tscircuit Contribution Tracker |
This PR prevents zero-length trace segments from being emitted when autorouter route data contains consecutive duplicate points.
Previously, convertHdRouteToSimplifiedRoute preserved adjacent same-layer duplicates such as A -> A -> B, which could later become a degenerate A -> A copper segment in exported PCB output. That kind of ghost segment can trigger downstream DRC issues even when the visible routing looks correct.