Skip to content

ipython console #220

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

ipython console #220

wants to merge 2 commits into from

Conversation

jf---
Copy link
Contributor

@jf--- jf--- commented Jul 1, 2025

having a console besides the viewer is a nice aid to help debugging or run demo's.
the

image

a neat feature of the console is auto reloading python modules in the background [1],
so you can keep coding in your IDE while not having to recreate a scene / relaunch a viewer on every chance.
(probably with a clever use of QTimer rerendering the code could also be invoked automagically)


there still is an issue with this code; when adding an object via the console, it does show up in the tree widget, but not in the renderer.

[1] https://ipython.org/ipython-doc/3/config/extensions/autoreload.html

jf--- and others added 2 commits July 1, 2025 09:50
remove superfluous `break`
neat for debugging demos and explorative coding, or making use of the autoreload function [1]

[1] https://ipython.org/ipython-doc/3/config/extensions/autoreload.html
@jf--- jf--- changed the title Jf/ipython console ipython console Jul 1, 2025
@tomvanmele
Copy link
Member

that is pretty cool indeed!

@tomvanmele
Copy link
Member

@Licini would be nice to make better use of the data store infrastructure of Qt and make the relevant widgets simply views of that store...

@Licini
Copy link
Collaborator

Licini commented Jul 8, 2025

@jf--- This is super cool! Just a heads up we will be doing a big clean up in terms how UI elements are hooked up (where we wrote a lot of spaghetti code in the past making lot of things unnecessarily complicated), if you were planning to make this one of the components, there is a draft PR to watch: #222

Meanwhile I guess it will also work just as a custom script for now.

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.

3 participants