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

add new standalone mode for startup #394

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

chmorgan
Copy link
Collaborator

Bringing this one back since it was helpful in debugging plugin changes that were resulting in crashes.

  • motivation: I wanted to be able to debug the UI with PyCharm/vscode
  • this is not the most stable thing in the world but gets the job done
  • allows the UI to start without Kicad being started, which allows real debugging from an IDE
  • see main.py for instructions on how to launch this
  • wrap all calls to GetBoard() with a class that can return a fake stub

@Bouni
Copy link
Owner

Bouni commented Jan 15, 2024

This does not affect a normal user, right?

@Bouni
Copy link
Owner

Bouni commented Jan 22, 2024

@chmorgan can you please rebase this one?

@Bouni
Copy link
Owner

Bouni commented Apr 2, 2024

@chmorgan Pleae rebase this one so I can test and merge it!

@chmorgan
Copy link
Collaborator Author

chmorgan commented Apr 2, 2024

@Bouni just saw this and the previous ask. Sorry for the delay will rebase this morning and repush.

@chmorgan
Copy link
Collaborator Author

chmorgan commented Apr 2, 2024

@Bouni with expanded use of pcbnew I'd like to clean this one up a bit, will resubmit this afternoon.

Presently it's a bit disjointed, and missing a pcbnew stub so pcbnew.GetSelected() doesn't work, and the naming is not as aligned as I'd like etc. This was highlighted by our more explicit use of pcbnew.XXX functions (a good thing imo) and expanded use of pcbnew functions.

@Bouni
Copy link
Owner

Bouni commented Apr 2, 2024

Sure, take your time!

@chmorgan chmorgan force-pushed the cmm_standalone branch 3 times, most recently from 8a73340 to 07d094e Compare April 3, 2024 01:49
@chmorgan
Copy link
Collaborator Author

chmorgan commented Apr 3, 2024

@Bouni alright, this is the reworked and cleaned up version. Could certainly use review and feedback.

@Bouni
Copy link
Owner

Bouni commented Apr 3, 2024

@chmorgan
Copy link
Collaborator Author

chmorgan commented Apr 3, 2024

@Bouni fixed the missing one, tests passing now.

Enables debugging of the plugin UI with PyCharm/vscode

- see __main__.py for instructions on how to launch
- Wrap KiCad calls with stubs

Based on the idea and initial PR authored by Dominic Cerquetti <binary1230@users.noreply.github.com>.
@Bouni
Copy link
Owner

Bouni commented Apr 10, 2024

Works on Windows 11 but gives an Error on startup:

grafik

If you click No it continues and starts the app.

@Bouni Bouni merged commit 663c7cc into Bouni:main Apr 10, 2024
2 checks passed
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.

None yet

2 participants