Skip to content

Add deterministic row layout for decoupling cap partitions#82

Open
hieu-lee wants to merge 2 commits into
tscircuit:mainfrom
hieu-lee:bounty-15-decoupling-cap-row
Open

Add deterministic row layout for decoupling cap partitions#82
hieu-lee wants to merge 2 commits into
tscircuit:mainfrom
hieu-lee:bounty-15-decoupling-cap-row

Conversation

@hieu-lee
Copy link
Copy Markdown

@hieu-lee hieu-lee commented May 11, 2026

Summary

  • add a focused decoupling_caps path in SingleInnerPartitionPackingSolver that bypasses generic packing
  • place caps in a deterministic centered row using decouplingCapsGap ?? chipGap and natural chip-id ordering
  • preserve fixed rotations and account for 90/270 degree dimensions when spacing
  • add regression tests for ordering, spacing, centering, and rotated dimensions

Demo

Verification

  • npx --yes bun@latest test tests/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.test.ts
  • npx --yes bun@latest run format:check
  • npx --yes bun@latest run build
  • git diff --check

/claim #15

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

@hieu-lee is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@hieu-lee
Copy link
Copy Markdown
Author

hieu-lee commented May 11, 2026

CI update: pushed 49be2a0 to avoid eager loading the debugger UI from the LayoutPipelineSolver06 fixture. GitHub test, format-check, and type-check are now passing on the PR. The only remaining red status is Vercel fork deployment authorization, which requires a tscircuit team member to authorize the deployment.

@hieu-lee
Copy link
Copy Markdown
Author

Demo artifact for the UI bounty review:

Short demo video: https://raw.githubusercontent.com/hieu-lee/matchpack/matchpack-82-demo/decoupling-cap-row-demo.mp4

Preview image:

deterministic decoupling-cap row demo

This visual was generated from the actual SingleInnerPartitionPackingSolver output for a partitionType === "decoupling_caps" fixture. Raw solver output used for the visual: https://raw.githubusercontent.com/hieu-lee/matchpack/matchpack-82-demo/decoupling-cap-row-demo.json

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.

1 participant