Skip to content

Fix : decoupling capacitor partition to include main chip#42

Open
Prashant27-07 wants to merge 5 commits intotscircuit:mainfrom
Prashant27-07:fix-decoupling-cap-layout
Open

Fix : decoupling capacitor partition to include main chip#42
Prashant27-07 wants to merge 5 commits intotscircuit:mainfrom
Prashant27-07:fix-decoupling-cap-layout

Conversation

@Prashant27-07
Copy link
Copy Markdown

@Prashant27-07 Prashant27-07 commented Mar 13, 2026

This PR fixes the decoupling capacitor partition logic in ChipPartitionsSolver.

Previously, partitions created for decoupling capacitor groups excluded the main chip.
This caused the partition to miss the chip that the capacitors were actually associated with.

This change ensures that the main chip is included in the partition together with the decoupling capacitors.

Closes #15

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 13, 2026

@Prashant27-07 is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@Prashant27-07
Copy link
Copy Markdown
Author

I ran the formatter and pushed the formatting commit.

The CI failure seems to come from a dependency issue:

convertCircuitJsonToSchematicSimulationSvg is not exported from circuit-to-svg.

The error appears unrelated to the changes in ChipPartitionsSolver.

Please let me know if you'd like me to investigate further or adjust the solver logic.

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.

Specialized Layout for Decoupling Capacitors

1 participant