Skip to content

align multi-component same-net rail chains to one coordinate - #918

Open
mohan-bee wants to merge 11 commits into
repro/audio-amplifier-tas2505from
fix/align-multi-component-same-net-rail-chains-v2
Open

align multi-component same-net rail chains to one coordinate#918
mohan-bee wants to merge 11 commits into
repro/audio-amplifier-tas2505from
fix/align-multi-component-same-net-rail-chains-v2

Conversation

@mohan-bee

@mohan-bee mohan-bee commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

motivation
trace cleanup should carry the first capacitor rail coordinate through an unlabeled unbranched component chain even when the last rail begins at a different coordinate.

before
cross-component rail discovery required both terminal rails to already match, so valid same-net chains could remain stepped instead of following the first capacitor rail.

before

after
the first terminal rail now anchors every following rail through the last capacitor; the tas2505 test compares all six rail coordinates with the real pre-alignment first-rail y, four affected snapshots are updated, and all 263 tests pass.

after

@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
schematic-trace-solver Ready Ready Preview Aug 26, 2026 4:19pm

Request Review

@mohan-bee
mohan-bee force-pushed the fix/align-multi-component-same-net-rail-chains-v2 branch from a705101 to f4fbda9 Compare August 26, 2026 15:36

@seveibar seveibar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see regressions

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