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

Kicad KLE plugin RGB support and merge conflicts #16

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

hksaperstein
Copy link

I took what @AlansCodeLog started wrt auto layout for RGB and addressed merge conflicts and minor execution bugs. I have not tested for specific rotation mode (wasn't too sure how to even set up my keyboard json for it), i did rotate the first switch in kicad and all other switches/components did follow. This also does force the user to use RGB, which is definitely not what is wanted. I have some work on an unpushed commit to use OO to allow for switch/components to be more encapsulated and selected more dynamically, but for now if this lives in a PR, at least another user could pull the commit for their rgb needs.

It also assumes that the footprints of all components have an origin at the center, but according to kicad docs, that seems to be the current standard/expectation anyway. last assumption is that all components start at ref 1, and D and LED are differentiated. Could be future room for selecting range of refs, but i think the component values could be set to LED, plugin executed, and changed back to D with new refs, without messing up the layout.

@hksaperstein hksaperstein changed the title Merge conflicts and LED support Kicad KLE plugin RGB support and merge conflicts Dec 8, 2024
@hksaperstein
Copy link
Author

if someone needs to use this commit, while still in PR, follow the original instructions for setting up the plugin and then basically copy this commit to the kle_placer_action.py file in the location of the plugin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants