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

jlcpcb has issues reading gerber files for 52 & 56 #13

Open
tsteffek opened this issue Oct 30, 2022 · 4 comments
Open

jlcpcb has issues reading gerber files for 52 & 56 #13

tsteffek opened this issue Oct 30, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@tsteffek
Copy link

I just wanted jlcpcb to make me some hillside52 PCBs, when I realized that they don't produce sensible prices. I checked the gerber view, and while it does present you with the board in the preview, the logging on the side tells you:

Analysis Results
layers : -1
minimum trace width : >=10 mil
minimum trace spacing : >=10 mil
minimum drill size : null
width : null
height : null
Unable to identify the layers due to the non-standard filename
Can not identify the board outline
...

I've tried with 56 for the same result and compared those to 46 and 48, which both seem to work (gets correct pricing, and Gerber Viewer doesn't complain about anything).

Disclaimer: This is my first time ordering PCBs and I have no idea what I'm doing, so while it's completely possible I messed up somewhere, I always thought downloading and uploading a .zip was fool-proof :D

@mmccoyd mmccoyd added the bug Something isn't working label Oct 30, 2022
@mmccoyd
Copy link
Owner

mmccoyd commented Oct 30, 2022

Sorry about that. I get the same result, which should not be the case.
I'll shift to making them the same way I make the 46 files, which will give you a PCBA option as well if you want it.
That might take a day or so.
Thanks for the issue report!

@tsteffek
Copy link
Author

That would be more than I ever dreamt of! Thank you.

@mmccoyd
Copy link
Owner

mmccoyd commented Nov 3, 2022

I pushed a fix for 52 to https://github.com/mmccoyd/hillside/releases.

The wiki is updated on how to order 52 with PCBA. It is now the same as for the 46.

Please let me know if that works for you! I've tested it with the JLC PCBA parts selection and preview steps for both front and back PCBA.

I've not fixed it for 56 yet.

@mmccoyd
Copy link
Owner

mmccoyd commented Nov 8, 2022

I'm fixing this for 56, which will also enable PCBA for it and put its Gerber and PCBA files on the releases page. Though its firmware will still be on a fork, not yet in QMK main. Then I'll use that to give 48 PCBA as well.

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