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

Bourns SDR1806 #543

Merged
merged 6 commits into from
Jul 8, 2018
Merged

Bourns SDR1806 #543

merged 6 commits into from
Jul 8, 2018

Conversation

bdm310
Copy link
Contributor

@bdm310 bdm310 commented May 2, 2018

Added the Bourns SDR1806 series footprint to Inductor_SMD.
https://www.bourns.com/docs/Product-Datasheets/SDR1806.pdf

image

KiCad/kicad-packages3D#305


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

@CLAassistant
Copy link

CLAassistant commented May 2, 2018

CLA assistant check
All committers have signed the CLA.

@poeschlr
Copy link
Collaborator

poeschlr commented May 6, 2018

The courtyard clearance should be 0.25mm
The body length is also not quite right. See annotated dimensioned drawing

screenshot from 2018-05-06 03-38-21

@bdm310
Copy link
Contributor Author

bdm310 commented May 6, 2018

Not sure where my 17.78mm came from and I think I did this before finding the 0.25mm spec in the KLC. I did 14.3+0.05 this time around; I'd say it's better to do the courtyard at max tolerance condition.

@Shackmeister
Copy link
Collaborator

Hi @bdm310 The angled sides of the Courtyard does still have 0,5mm clearance.
I believe it should be ready to merge after this has been fixed

@bdm310
Copy link
Contributor Author

bdm310 commented Jun 26, 2018

I must've done it to the silk and not the body. Updated.

@Shackmeister
Copy link
Collaborator

getting closer :)
please change silkscreen width to 0.12mm instead of 0.15mm
also please move the REF and Value closer to the courtyard :)

@bdm310
Copy link
Contributor Author

bdm310 commented Jul 6, 2018

I wasn't consistent (one of the arcs was 0.12) so this would've needed an update anyway but it may be helpful to clarify in the KLC that 0.12 should only be deviated from if necessary. The current wording makes it seem relatively loose and I'd call this a "low density" part. It'd also definitely be helpful to include a guideline for where reference designators and values should go.

@Shackmeister
Copy link
Collaborator

@bdm310
Thanks for the contribution :)
hope to see more :)

@Shackmeister Shackmeister merged commit bad2420 into KiCad:master Jul 8, 2018
@myfreescalewebpage myfreescalewebpage added the Addition Adds new footprint to library label May 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Addition Adds new footprint to library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants