Skip to content

feat: honor footprint attributes of current variant - #245

Open
piernov wants to merge 1 commit into
bennymeg:masterfrom
piernov:feat/footprint-variant
Open

feat: honor footprint attributes of current variant#245
piernov wants to merge 1 commit into
bennymeg:masterfrom
piernov:feat/footprint-variant

Conversation

@piernov

@piernov piernov commented Jun 17, 2026

Copy link
Copy Markdown

Add support for variants introduced in KiCad 10.

Not thoroughly tested in KiCad 10, and not tested at all in KiCad <= 9.

@piernov
piernov force-pushed the feat/footprint-variant branch 3 times, most recently from 9d531a0 to 1fb8682 Compare June 17, 2026 18:25
@piernov
piernov force-pushed the feat/footprint-variant branch from 1fb8682 to e4d29b0 Compare June 17, 2026 18:57
@silbe

silbe commented Jul 9, 2026

Copy link
Copy Markdown

(I'm not a maintainer, just another user)

Haven't checked each change individually against the API docs but the changes look sensible and complete. The one thing I'm not sure about (and cannot currently try out) is exactly how KiCad v10 represents footprints that are variants of the same symbol and whether that's handled correctly in the plugin. I'm guessing they are represented as independent footprints with the same reference designator and that it triggers the "unique designator" logic. In that case it may be useful to ignore footprints that are going to be excluded while counting the footprint designators in generate_tables() (otherwise the designators in the production files will be different from the ones in the KiCad files even though for each variant the designator is unique).

Tested successfully with v9: When used from a) within the KiCad v9 PCB editor and b) on the command line, both times on a moderately complex design (hierarchical design with several instances of the same sheet) it produces unchanged output (except for the timestamps included in the files) and I don't see any error message or warning on the command line.

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.

2 participants