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

Seiko ms621 f #1218

Merged
merged 16 commits into from
Jan 20, 2019
Merged

Seiko ms621 f #1218

merged 16 commits into from
Jan 20, 2019

Conversation

Misca1234
Copy link
Collaborator

@Misca1234 Misca1234 commented Dec 26, 2018

When created the 3D model for battery holder Seiko MS621F
https://www.sii.co.jp/en/me/files/2014/02/file_EXTENDED_PRDCT_SPEC_75_FILE_11.jpg

It seems that the battery circles was off 0.4, earlier origo was at -4.6 but should be 5.0

Foot print push
#1218

3D model push
KiCad/kicad-packages3D#486

3D model script push
easyw/kicad-3d-models-in-freecad#245

bild

bild

bild


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

@poeschlr poeschlr added Addition Adds new footprint to library Pending reviewer A pull request waiting for a reviewer labels Dec 29, 2018
@poeschlr
Copy link
Collaborator

The courtyard clearance is not quite right.

The land pattern it self is ok.

screenshot from 2019-01-16 22-07-18
screenshot from 2019-01-16 22-02-09

@poeschlr poeschlr self-assigned this Jan 16, 2019
@poeschlr poeschlr added Pending changes and removed Pending reviewer A pull request waiting for a reviewer labels Jan 16, 2019
@Misca1234
Copy link
Collaborator Author

Move the courtyard a little bit out

bild

bild

@poeschlr poeschlr added Ready for review Use this to mark pull requests that are updated but you could not review instantly and removed Pending changes labels Jan 17, 2019
@poeschlr
Copy link
Collaborator

The bottom side connection between the arc and the horizontal line is open (by more than 0.01mm which is the maximum tolerance for the DRC algorithm to work.)

@poeschlr poeschlr added Pending changes and removed Ready for review Use this to mark pull requests that are updated but you could not review instantly labels Jan 17, 2019
@Misca1234
Copy link
Collaborator Author

Circle end have been moved

@poeschlr poeschlr added Ready for review Use this to mark pull requests that are updated but you could not review instantly and removed Pending changes labels Jan 20, 2019
@poeschlr
Copy link
Collaborator

looks good now. Thanks

@poeschlr poeschlr merged commit 2589e9c into KiCad:master Jan 20, 2019
@Misca1234 Misca1234 deleted the Seiko_MS621F branch January 21, 2019 05:51
DaToBSn pushed a commit to DaToBSn/kicad-footprints that referenced this pull request Jan 21, 2019
* 'master' of github.com:KiCad/kicad-footprints: (39 commits)
  Crystal: Add Abracon ABS25 4Pin 8.0x3.8mm (KiCad#1270)
  RJ45: Add a new Amphenol RJHSE5380-08 footprint (KiCad#1300)
  Add DFN-10 with EP 1.2x2 (KiCad#1290)
  Seiko ms621 f (KiCad#1218)
  Sip package link fixes (KiCad#1215)
  RF Module: Add CMWX1ZZABZ (KiCad#1296)
  Add L_Wuerth_WE-PD-Typ-7345. (KiCad#1299)
  Converter_DCDC XP_POWER IA, IH, ITX and ITQ series (KiCad#1058)
  More 3M Textool ZIF sockets, generated with script. (KiCad#495)
  Changed courtyard clearance to 0.25mm.
  Added Kyocera 6 pin tcxo.
  Add JST PHD series of connector footprints (KiCad#915)
  Rename Cherry MX1A to Cherry MX (KiCad#1180)
  RF_Module: Fix and improves HOPERF_RFM9XW_SMD footprint (KiCad#1213)
  Add vertical USB Type-C DX07S024WJ1 (KiCad#1251)
  Add PAT1220 RF attenuator. (KiCad#1288)
  Add RJ45_BEL_SS74301-00x (KiCad#1036)
  Add Texas R-PUQFN-N10 (KiCad#1310)
  Add LairdTech_28C0236-0JW-10 Ferrite bead (KiCad#1040)
  Fix courtyard and silkscreen
  ...
@poeschlr poeschlr removed the Ready for review Use this to mark pull requests that are updated but you could not review instantly label Feb 4, 2019
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.

2 participants