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

Correct JST MPNs #1238

Merged
merged 1 commit into from
Feb 9, 2019
Merged

Correct JST MPNs #1238

merged 1 commit into from
Feb 9, 2019

Conversation

evanshultz
Copy link
Collaborator

Fixes #911

This all started from #1141 and I documented the first couple issues at #911. I've now looked at all JST connector footprints and fixed only the errors in MPN and make other non-breaking updates (fixed keywords, etc.).

General single-row connectors:

  • Changed "side entry" to "horizontal" and "top entry" to "vertical" in keywords

ACH (http://www.jst-mfg.com/product/pdf/eng/eACH.pdf):

  • MPN started with "BBM" and should be "BM"

EH (http://www.jst-mfg.com/product/pdf/eng/eEH.pdf):

  • When the pin count is <10 there is no leading zero in the MPN
  • There is no 16 pin version

J2100 (http://www.jst-mfg.com/product/pdf/eng/eJFA-J2000.pdf, page 7):

  • Keying was not in the footprint name; this is unclear (and affects 3D models) so could somebody check me on this?
  • Changed pin numbers a1 through b5 to 1 though 10 (the "Line No" in the datasheet is not related to the pin number)

PH (http://www.jst-mfg.com/product/pdf/eng/ePH.pdf):

  • Abstracts away the exact PN for normal vs low insertion force THT headers, but I left it alone

SFH (http://www.jst-mfg.com/product/pdf/eng/eSFH.pdf):

  • Corrected MPN

XH (http://www.jst-mfg.com/product/pdf/eng/eXH.pdf):

  • When the pin count is <10 there is no leading zero in the MPN
  • Removed JST_XH_B11B-XH-AM_1x11_P2.50mm_Vertical (doesn't exist)

ZE (http://www.jst-mfg.com/product/pdf/eng/eZE.pdf):

  • For SMT, the top entry was massing the final "T" and the side entry had a final "T" but shouldn't have

The following appear to be OK to be in footprint naming, so just keyword updates: AUH, GH, JWPF, LEA, NV, PH, SHL, SH, SUR, VH, XAG

I used Python 3.7.0.

The number of additions and deletions seems odd to me, but I'm quite sure I was careful with only updating the changed files so I think this is correct.

@poeschlr
Would you be willing to correct the EH and XH 3D models? Those are the only ones at https://github.com/KiCad/kicad-packages3D/tree/master/Connector_JST.3dshapes that are affected and it looks like you submitted the existing files. Since you're not busy with anything else I hope that's OK. :)


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

@evanshultz
Copy link
Collaborator Author

Script update is at pointhi/kicad-footprint-generator#256.

Travis shows a red 'x' above but the Details page is all green that I can see. Am I doing something wrong? Maybe the quantity of file changes is too great for Travis to handle and it bails?

@poeschlr
Copy link
Collaborator

The travis error comes from the library table check. Looks like i missed the problem in the pull request i merged.

@poeschlr poeschlr added Bug Fix footprint existing in the library Pending reviewer A pull request waiting for a reviewer labels Dec 30, 2018
@poeschlr
Copy link
Collaborator

poeschlr commented Dec 30, 2018

At least JST EH,PH and XH also have 3d models in our lib. Meaning you would need to also update these scripts and the 3d models in our lib.
Edit: Ok just noticed you already gave this detail above.

@evanshultz
Copy link
Collaborator Author

@poeschlr
Is there a fix that's needed for the table check?

Would you be willing to do the 3D models? Or @Shackmeister or @Misca1234 ?

@poeschlr
Copy link
Collaborator

The table stuff is already fixed in a pull request of mine.

@evanshultz
Copy link
Collaborator Author

Thanks, I looked but didn't catch it right away. All is green now with Travis.

@poeschlr poeschlr self-assigned this Feb 8, 2019
poeschlr added a commit to poeschlr/packages3D that referenced this pull request Feb 8, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented Feb 9, 2019

Thanks, with the 3d models now ready this can be merged.

@poeschlr poeschlr merged commit 2b80fc8 into KiCad:master Feb 9, 2019
@poeschlr poeschlr removed the Pending reviewer A pull request waiting for a reviewer label Feb 9, 2019
@evanshultz evanshultz deleted the fix-jst-pns branch February 9, 2019 14:24
DaToBSn pushed a commit to DaToBSn/kicad-footprints that referenced this pull request Jul 2, 2019
* 'master' of github.com:KiCad/kicad-footprints: (38 commits)
  Reexport texas qfn footpritns with updated scripts (KiCad#1358)
  Scripted pwqfn-n24 for BQ25601 (KiCad#1402)
  Scripted TI S-PWQFN-N16 EP2.1 for motor drivers (KiCad#1401)
  New scripted version of TI footprints (KiCad#1360)
  Add QFN-64-1EP_8x8mm_P0.4mm_EP6.5x6.5mm (KiCad#1405)
  Added footprint for RECOM RPM-series of DC/DC converters (1A, 2A, 3A and 6A). (KiCad#1382)
  add new vertical fuseholders (KiCad#1326)
  Add Molex 200528 series (scripted via kicad-footprint-generator) (KiCad#1383)
  Button_Switch_THT: Add CK OS102011MS2QN1 footprint (KiCad#761)
  Add SPST relay TE PCN-1xxD3MHZ (KiCad#1357)
  Added battery holder BatteryHolder_TruPower_BH-331P_3xAA (KiCad#1359)
  Added SIM28ML footprint (KiCad#1355)
  add Fischer FK244_13_D2_PAK SMD heatsink (KiCad#1217)
  Connector: Add B6A-PCB-45 from IHI connectors (KiCad#1085)
  Correct JST MPNs (KiCad#1238)
  Rename SIP-3 Packages in Sensor_Current (KiCad#1072)
  Add D52M (KiCad#1117)
  Fixed nonsenses in the 7seg footprint
  Fix Cherry and Matias keyswitch Fab line thickness (KiCad#1376)
  Add Coilcraft LPS5030 (plus freecad source) (KiCad#1340)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Fix footprint existing in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JST connectors have wrong PN
2 participants