-
Notifications
You must be signed in to change notification settings - Fork 109
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
Develop #3
Develop #3
Conversation
…t to QRectF conversion as is done elsewhere in the code already
…adConfig instead of filePath
previous code only allowed overwrite of existing files, no new files could be created
…teractively creating connections
fair enough about running inside an environment that has already supplied a QApplication and an event loop... but shouldn't the demo simply run from the command line by doing "python nodz_demo.py" |
already has a QApp+event loop
OK... so I've addressed the concern by doing a try/except on the creation of the QApp and only do an exec if an app was created |
Just to add my voice to this - I hit these exact same issues when trying to run from the cli. These changes resolved the issue. Maya isn't listed as a prerequisite - so if it's expected that Maya is in the mix, then it should really be added to the docs, but making nodz available to standalone devs would have a great deal more value in my mind. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed a variable typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaced QRect by QRectF
had a few minor problems trying out the demo code... here are my fixes for those issues