Skip to content
This repository has been archived by the owner on Nov 21, 2017. It is now read-only.

Added JAE LY Angled connectors #3

Closed
wants to merge 2 commits into from
Closed

Added JAE LY Angled connectors #3

wants to merge 2 commits into from

Conversation

SchrodingersGat
Copy link
Contributor

This PR adds the complete series of LY angled connectors

Datasheet
http://www.jae.com/z-en/pdf_download_exec.cfm?param=SJ038187.pdf

@pointhi
Copy link
Collaborator

pointhi commented Jul 24, 2016

Please check Dimension B. If you are interested about a real top view, you also have to consider Dimesion C.

the rest looks fine

@SchrodingersGat
Copy link
Contributor Author

Can you provide a bit more info as to what you mean here? :) Happy to adjust but not quite sure where the issue is...

@evanshultz
Copy link
Collaborator

@pointhi Can you share more about what you found so we can complete this PR? Thanks!

@poeschlr poeschlr closed this Jun 22, 2017
@poeschlr poeschlr reopened this Jun 22, 2017
@SchrodingersGat
Copy link
Contributor Author

KLC issues have been addressed

@evanshultz
Copy link
Collaborator

@pointhi @poeschlr @jkriege2 Is someone able to review this?

@poeschlr
Copy link
Collaborator

Am i right in assuming these footprints are script generated? (Makes checking easier.)

The pin order might not be the best choice for the symbols we currently have. (odd/even would be better such that we can use the current symbols.)
In the datasheet pin names are given similar to BGA pin numbers but with letter and number exchanged (1A, 2A, .... nA; 1B, 2B, ..., nB) See issue for a discussion if we should create such symbols KiCad/kicad-library#1451

Pin 1 marker on fab layer is missing.

I assume @pointhi meant the following with his mention of dimension b and c (I don't think this is important/ we can merge without changes to it.):
screenshot from 2017-07-26 23-48-15

@poeschlr poeschlr self-assigned this Jul 26, 2017
@evanshultz
Copy link
Collaborator

@SchrodingersGat Can you review Rene's comments above, please?

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.

4 participants