Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Add initial support for custom pad shapes #232

Merged
merged 1 commit into from
Apr 29, 2018

Conversation

Ratfink
Copy link
Collaborator

@Ratfink Ratfink commented Mar 31, 2018

This PR adds initial support for custom pad shapes in the KicadMod class. This does most of the work described in #231. The KicadMod class can now load all custom pad primitives. It can also check the bounding box of all custom pad primitives except for arcs, which will be more complicated to implement than the others. Since there are no footprints in kicad-footprints yet that use arc primitives (or in fact, any primitives except for polygons) in custom pads, I don't think this is a problem yet. 🙂

There is no support for exporting custom pad information yet either.

This commit adds initial support for custom pad shapes in the KicadMod
class.  The class can now load all custom pad primitives.  It can also
check the bounding box of all custom pad primitives except for arcs,
which will be more complicated to implement than the others.  Since
there are no footprints in kicad-footprints yet that use arc primitives
in custom pads, I don't think this is a problem yet. 🙂

There is no support for exporting custom pad information yet.
@poeschlr
Copy link
Collaborator

I can not see anything obviously wrong.
I assume you tested it locally? (I will merge as soon as you confirmed that.)

@Ratfink
Copy link
Collaborator Author

Ratfink commented Apr 23, 2018

Yes, I tested it locally. It correctly processes KiCad/kicad-footprints#461, and a test footprint I kept adding random primitives to while writing this.

@evanshultz
Copy link
Collaborator

@poeschlr
Can this be merged then?

@poeschlr
Copy link
Collaborator

Oh yes. sorry forgot about that.

@poeschlr poeschlr merged commit ce96cce into KiCad:master Apr 29, 2018
@Ratfink Ratfink deleted the custom-pads branch April 29, 2018 01:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants