Skip to content

Keep net labels on corners when wires can shift clear - #1117

Open
mohan-bee wants to merge 6 commits into
mainfrom
fix-rail-net-label-corners
Open

Keep net labels on corners when wires can shift clear#1117
mohan-bee wants to merge 6 commits into
mainfrom
fix-rail-net-label-corners

Conversation

@mohan-bee

@mohan-bee mohan-bee commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Motivation

  • Keep net labels on their existing trace corners when neighbouring wires can shift clear.

Before

  • A movable wire could reject a valid corner placement and produce an unnecessary offset label connector.

After

  • Reuse the existing overlap solver to validate a straight outward wire shift, then let the existing collision stage apply it.
  • The RTC/IMU/microSD and 20260804T095800Z snapshots improve with no new final-output label collisions; all 348 tests, type checking, and the build pass.

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
schematic-trace-solver Ready Ready Preview Sep 11, 2026 6:08pm UTC

Request Review

@mohan-bee mohan-bee changed the title Snap rail net labels to connector corners Align detached rail labels beside host corners Sep 11, 2026
@mohan-bee mohan-bee changed the title Align detached rail labels beside host corners Place detached rail labels on a shared trace corner Sep 11, 2026
@mohan-bee mohan-bee changed the title Place detached rail labels on a shared trace corner Place rail labels at corners without connector detours Sep 11, 2026
@mohan-bee mohan-bee changed the title Place rail labels at corners without connector detours Keep net labels on corners when wires can shift clear Sep 11, 2026
Base automatically changed from repro-rtc-imu-microsd-traces to main September 11, 2026 19:05
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.

1 participant