-
Notifications
You must be signed in to change notification settings - Fork 41
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
rotation source and wildcards/regex in overrides #1
Comments
Thanks! Here is a proposed solution:
WDYT? |
Great idea, much better than what I was thinking. JLCPCB has the data because they display a preview - I'll talk to them and see if they will assist. |
JLCPCB was helpful, but says that they don't have information as to how each part is oriented on tape. But package (eg, SOIC8) is available for each part and I see that others do regex matching on footprint to determine the rotation needed transform from tape to the kicad standard. Not clear to me how reliable this is - but it may be the best option. |
Wery nice! With regard to kicad_pos_to_cpl.py: I like the simplicity of your kicad_pos_to_cpl.py script (cant say anything about BOM as I use kibom-master with customl jlc-kibom.ini to generate JLCPCB bom and using [JOIN] to to accumulate/rename columns), however a few small additions to your kicad_pos_to_cpl.py might be useful:
Maybe additional option for negating x axis for bottom side (it seems to me that kicad .pos is inverted for bottom side but JLC requires it as coordinates would look from "side of components" - am i correct?), Thanks, R. |
+1 for package idea - I made that modification (in the simplest way - no REs etc) and it seemed quite effective. |
Nice program, thanks. Looks like I have to rotate all capacitors and resistors, so pattern matching (like C*) in overrides.json would be convenient. It's too bad that rotation information is stored in a separate file and not tied to a specific part - preferable would be to store rotation information in a field in the schematic/part (which can be available as a BOM field).
The text was updated successfully, but these errors were encountered: