Closed general

kicad-lane: current_capacity lists a pour layer once per pour (poursOn repeats)

Colby Knox · 14d ago ·closed by Colby Knox

KiCad bridge 1.0.67 vendors adom-eda-checks 0.1.2 (standard 0.4.3, commit 22c6ac7b) unchanged and answers kicad_copper_balance, kicad_return_path, kicad_impedance and kicad_current_capacity through it.

One thing in the engine: current_capacity lists a layer in poursOn once for each pour of the net on that layer. In current.rs, check() collects it from polygons without dedup.

  • KiCad's StickHub demo, {"nets":{"+5V":1.5}}: "poursOn": ["B.Cu", "B.Cu"].
  • The routed ESC G431 board, GND: "poursOn": ["F.Cu","F.Cu","F.Cu","F.Cu","F.Cu","F.Cu","B.Cu","F.Cu","B.Cu","F.Cu","B.Cu","In1.Cu"].

Expected: each layer once, e.g. ["B.Cu", "F.Cu", "In1.Cu"].

I have not touched the vendored copy, and I'll pick up the fix at your next engine tag.

Reproduce with the engine alone: a board whose polygons has two {"net": "VIN", "layer": "B.Cu"} entries, and args.nets = {"VIN": 1}.

1 Reply

Colby Knox · 14d ago

Fixed in standard 0.4.5, engine 0.1.4 (48925b7). poursOn lists each layer once however many pours the net has there. The test now has two B.Cu pours and expects ["B.Cu"]. Thanks for not patching the vendored copy.

Take 0.4.5 at your next re-vendor. It also carries 0.1.3's fix: min_clearance counts only copper-to-copper clearance rules. Closing.

Colby Knox closed this · 14d ago

Log in to reply.