-
Notifications
You must be signed in to change notification settings - Fork 716
Conversation
add Alpha 9 mm vertical pots
Please review. |
Just from the screenshot alone i can already tell you that these are missing pin 1 markers on both silk and fab layers. I assume the bottom ones are circular hole versions of the top ones. I am not so sure the ones on the bottom right have enough pad to pad clearance to be of any use. It might simply be that we can not offer such an option for every part. |
@robkam, just because I've seen it in the commits in this pull request, will you submit the Alpha pots too? It would be nice :) Thanks! |
The Alpha pots are at #1073 (I couldn't figure out how to do a seperate PR for these). |
Thanks, I'll comment on that PR. |
I left out pin 1 markers because these're non polarised. There's a few footprints already in Button_Switch_THT without pin 1 markers on silk/fab layers, e.g. the similar SW_NKK_G1xJP Could I have same clarification please. Yes the bottom ones are the circular hole versions. Included because the holes are cheaper to have produced. The clearance is 0.45 mm between the sub-mini circular pads, at their closest. |
To make separate pull requests you need to have one branch per contribution. I am assuming you have the official library repo added as a second remote and called it upstream: git fetch upstream # get the state of the official repo
git checkout upstream/master # change to the current head of the official repo
git checkout -b new_sensible_branch_name # create a new branch from that state
git push --set-upstream origin new_sensible_branch_name # push that new branch to github
# after you made your changes work normally with
git add -u # or git add list_of_files
git commit # enter sensible commit message as usual
git push # push your changes online if you then are requested to do some changes you would need to checkout the correct branch and then do your changes. git checkout branch_connected_to_pull_request
# do your changes
git add -u # or again git add file_list
git commit # detail what you changed in the commit message
git push # push your fixes to github (will automatically appear in the pull request) To add the official repo as a second remote called upstream (one time setup): git remote add upstream https://github.com/KiCad/kicad-footprints.git |
Thanks that's useful :-) ... except I'm doing it all through a browser via GitHub. |
The browser workflow has its limitations. So it might be time to start looking into having a local git clone. |
Regarding workflow: I would guess that even the browser workflow offers some way to make a new branch for every pull request you open. But i have never used the browser for anything as it is simply way more work that way. |
I'll look at the command line as I seem to have got myself stuck with the browser. |
There are git tools with graphical interfaces if you would feel more at ease that way. But the only thing that is truly universal is the command line. |
I'd find the command line easier, once I'm used to it. I'd forgotten that I have Git GUI to hand, I'll have a look at it ... |
I started to look at this but didn't have time to get anything written down before something else came up. However, I will note that these will need the manufacturer name on them or some real definition about what type of switch fits in this footprint. Even if there are multiple switches that could be so you don't want to be too specific, the current naming is way too generic. |
Something like |
I'm not sure that the switch configuration matters for these switches. One footprint could support a variety of switch configurations. For https://www.ckswitches.com/media/1394/7000toggle.pdf how about:
And for https://www.ckswitches.com/media/1391/ttoggle.pdf how about:
I recall a discussion about switch names which I may not have found just now, so there might be some tweaks. But that ought to at least be in the ballpark. #759 and KiCad/kicad-symbols#580 (even though it's about symbols there are lots of linked issues and PRs) are good places to look. Edit: Switch orientation, for example, may be wise to catch. But the above at least directs someone towards a relevant part. I didn't mean the above is mutually exclusive with more information. |
The switch configuration matters if we want to have better footprint filters on the symbol side. (I assume with switch config you mean things like normally open, number of possible positions, ...) Edit: I would also guess that some other parameters would matter for the 3d model side. And yes footprint aliases would really be nice. |
Then: |
Ahh, right. Thanks, Rene. If there are no options besides |
The idea behind including Lever (and other such options) is such that similar switches are grouped together. To be honest it is kind of an experiment. I would suggest that we keep the current naming scheme for a while and see how it develops. If it is a hindrance to what we want to achieve then we can look at it again and see if we can find something better for v6. |
add vertical THT C&K toggle switches, SPST, SPDT and DPDT, miniature & sub-miniature
Mini https://www.ckswitches.com/media/1394/7000toggle.pdf
Sub-mini https://www.ckswitches.com/media/1391/ttoggle.pdf