From 0f98a928be50672fabfd3db3f406a9b876caf652 Mon Sep 17 00:00:00 2001 From: oceanlight-cn Date: Thu, 2 Apr 2026 11:53:17 +0800 Subject: [PATCH] fix: use shortest_connection_along_outline for decoupling capacitors layout Changes the packing strategy for decoupling capacitor partitions from 'minimum_closest_sum_squared_distance' to 'shortest_connection_along_outline' to create a clean, linear arrangement instead of scattered placement. Addresses Issue #15 --- .../SingleInnerPartitionPackingSolver.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/solvers/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.ts b/lib/solvers/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.ts index 88db103..b6c56a5 100644 --- a/lib/solvers/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.ts +++ b/lib/solvers/PackInnerPartitionsSolver/SingleInnerPartitionPackingSolver.ts @@ -129,15 +129,20 @@ export class SingleInnerPartitionPackingSolver extends BaseSolver { }) let minGap = this.partitionInputProblem.chipGap + let packPlacementStrategy = "minimum_closest_sum_squared_distance" + if (this.partitionInputProblem.partitionType === "decoupling_caps") { minGap = this.partitionInputProblem.decouplingCapsGap ?? minGap + // Use shortest_connection_along_outline for decoupling caps to create clean linear layout + // instead of scattered placement (addresses issue #15) + packPlacementStrategy = "shortest_connection_along_outline" } return { components: packComponents, minGap, packOrderStrategy: "largest_to_smallest", - packPlacementStrategy: "minimum_closest_sum_squared_distance", + packPlacementStrategy, } }