Skip to content

Improve net-aware decoupling cap layout#102

Open
stainlu wants to merge 1 commit into
tscircuit:mainfrom
stainlu:codex/net-aware-decap-layout
Open

Improve net-aware decoupling cap layout#102
stainlu wants to merge 1 commit into
tscircuit:mainfrom
stainlu:codex/net-aware-decap-layout

Conversation

@stainlu
Copy link
Copy Markdown

@stainlu stainlu commented May 12, 2026

Summary

  • preserve allowed VCC/GND net membership when decoupling-cap-only partitions inherit those nets through strong connections to the main chip
  • add a deterministic fast path for partitionType === "decoupling_caps" that orders caps by main-pin coordinates and spaces by the full body + pin envelope
  • prefer the positive-voltage-connected external main pin for ordering when both cap pins are strongly connected, with deterministic natural-id fallback behavior
  • pin circuit-to-svg@0.0.345 so the existing full Bun test suite can load the schematic simulation SVG export

/claim #15

Verification

  • npx --yes bun test tests/ChipPartitionsSolver.test.ts tests/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.test.ts
  • npx --yes bun test
  • npx --yes bun x tsc --noEmit
  • npx --yes bun run format:check
  • npx --yes bun run build
  • git diff --check

AI-assisted with Codex; I reviewed the patch and verification locally.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

@stainlu is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Member

@imrishabh18 imrishabh18 left a comment

Choose a reason for hiding this comment

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

@stainlu We have stopped issuing bounties, but we do sponsor people for the contributions (refer our docs for more info).

You need to add test with visualization snapshots for me to know if this even works!

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.

2 participants