Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Too small PCB holes for Choc v1 #21

Open
bolleywall opened this issue Sep 8, 2023 · 2 comments
Open

Too small PCB holes for Choc v1 #21

bolleywall opened this issue Sep 8, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@bolleywall
Copy link

In marbastlib Choc v1 footprints, the diameter of the holes for the smaller mounting pins is 1.7 mm., while Kailh recommends 1.9 mm.

@ebastler ebastler self-assigned this Sep 8, 2023
@ebastler ebastler added the bug Something isn't working label Sep 8, 2023
@ebastler
Copy link
Owner

ebastler commented Sep 8, 2023

Hi! Thank you for pointing this out.

If I remember correctly we originally opted for this to allow plateless hotswap to work decently without wobbling, and then mistakingly used the footprint for both HS and solder.

It is tested and works (although the switches will afterwards probably sit loosely in 1.9mm hole PCBs if you remove them from ours). I am considering to keep it for hotswap, but switch to the recommended hole sizes for solder footprints as those do not need the mechanical stability of tight legs.

On the other hand I feel reluctant to change something tested and working. I will order a testing PCB with my next PCB order (with footprints with 1.7mm, 1.8mm and 1.9mm) and decide after comparing them side-by-side.

@bolleywall
Copy link
Author

Thank you for explaining your reasoning! Now I too need to order a test PCB and compare various hole sizes for both the side posts and the middle one :-)

Apparently some feel 1.9 mm is too large (link for reference: https://old.reddit.com/r/ErgoMechKeyboards/comments/mdvy5w/what_size_through_holes_do_you_use_for_kailh_choc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants