Skip to content

fix: coalesce outline grid coordinates to prevent board109 slivers - #166

Merged
ShiboSoftwareDev merged 3 commits into
repro/board109-outline-sliversfrom
fix/board109-outline-slivers
Oct 21, 2026
Merged

ShiboSoftwareDev merged 3 commits into
repro/board109-outline-sliversfrom
fix/board109-outline-slivers

Conversation

@ShiboSoftwareDev

@ShiboSoftwareDev ShiboSoftwareDev commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Coalesce near-identical outline grid coordinates to remove redundant blockers around board109. Stacked on #165 and used by tscircuit/tscircuit-autorouter#3144. Validation: 112 tests pass, 1 existing skip; build, type checking, and formatting checks pass.

Board109 outline Before After
Void rectangles 19,985 284
Floating-point slivers 14,696 0

Native seeding visualize() with 0.3 mm board-edge clearance; darker areas show overlapping blockers.

Before After
Board109 native seeding visualization before the fix Board109 native seeding visualization after the fix

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
rectdiff Ready Ready Preview Oct 20, 2026 4:35pm UTC

Request Review

@0hmX

0hmX commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

i think i get it just better visaltion could be good; no idea tho what can be better

@ShiboSoftwareDev
ShiboSoftwareDev added this pull request to stack #168 October 21, 2026 16:44
@ShiboSoftwareDev
ShiboSoftwareDev merged commit 4cbc6e4 into main Oct 21, 2026
5 of 6 checks passed
@tscircuitbot

Copy link
Copy Markdown
Contributor

Thank you for your contribution! 🎉

PR Rating: ⭐⭐⭐
Impact: Major

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


This branch was successfully deployed

1 active deployment
Preview — 00ea18bc Deployed Oct 20, 2026 by vercel[bot]
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.

3 participants