-
Notifications
You must be signed in to change notification settings - Fork 710
Add LT/ADI BGA/LGA-133 uModule footprints #1027
Conversation
Have you been thinking of making these with the scripts ? We should be able to modify the BGA script to make |
Actually, I didn't even think of it. But I should have. Since there are still many hand-built footprints that could be included in a script I hope this is OK to merge. You are correct that we should go through the library and migrate hand-built footprints to scripted ones at some point. |
Go with the data sheet I can check the manually ones, but hint, hint
I will checkt he squared one tonight, i will also check the python script to see if they can handle squared pads, if not, I will add the possibility |
OK. Thanks. It cannot do square pads. I made the last commit to the script so I'm a bit familiar with Clay's work. I also just realized that it doesn't check that the length of the row_skips <= layout_y and the length of each element in row_skips is <= layout_x. That would be one way to validate input data and to help catch potential user input errors. It also doesn't use the nice stream feature for input data like Rene's DFN/QFN generator and its spawn. I was thinking to make that change and move this generator with rest of the "Package_" generators once I got a few more things off my plate, but if you want to do any of it that would be great too. Go for it. :) |
No, I saw that too, but I think this script is fairly old, I guess it pre-dates the discussion of what "input method" should be used, please go ahead and modify the script with checks. Just add a paramter which can be set to "circle", "rect and "roundrect", then we should cover many situations like this |
Ok, then I modify the script to handle circle, rect and roundrect pads. Do you want to wait for the change to be merged (bga.py) and re generate your foot prints |
Here is the script push |
@evanshultz |
Thanks for updating the script! Since the script shouldn't generate footprints any different than these I'm still happy to merge them as is. |
Then we go with this |
https://www.analog.com/media/en/technical-documentation/data-sheets/4637fc.pdf
Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items: