Skip to content

Add THT non-polar caps

evanshultz requested to merge github/fork/evanshultz/c-np-tht into master

As noted at https://github.com/KiCad/kicad-footprints/issues/900, we didn't have non-polar electrolytic cap footprints.

http://kicad-pcb.org/libraries/klc/F3.3/ specifies something different than the existing footprints names: THT caps don't have height in the footprint name even though it is known (the 3D model has a height) so our footprints are really missing information right now. Putting the footprint selection in CvPcb for a generic part like this means the user doesn't have to choose the height until the schematic is finished. It is a bit of a burden, but otherwise aren't we making decisions (picking capacitor height) on behalf of the user which is worse? I think height and pitch should be required attributes for radial caps. Perhaps in trying to combine the different body styles this section of KLC isn't clear enough and it should be separated by body style?

I could update the existing footprints to include the above and also match what I believe KLC is saying but I'm not sure if re-naming footprints is allowed right now so I left it all alone.

Script PR: https://github.com/pointhi/kicad-footprint-generator/pull/200

Nonpolar datasheets used:

image

Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items:

  • Provide a URL to a datasheet for the footprint(s) you are contributing
  • An example screenshot image is very helpful
  • If there are matching symbol or 3D model pull requests, provide link(s) as appropriate
  • Check the output of the Travis automated check scripts - fix any errors as required

Merge request reports