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

Added additional debugging output to pyface/qt/__init__.py. #1230

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

Conversation

capn-freako
Copy link

Responding to issue 1205, by adding some additional debugging output when importing a GUI backend fails.

@capn-freako
Copy link
Author

Fixes #1205

@capn-freako
Copy link
Author

Here is example output from the new code, in the case of import failure:

ImportError: Nothing found in sys.modules or os.environ.
Attempting to import...
Successfully imported PySide2
Failed to import QtCore; problem involved: C:\Users\Admin\miniconda3\envs\pybert\lib\site-packages\PySide2\QtCore.cp39-win_amd64.pyd.
Failed to import PySide6; problem involved: None.
Successfully imported PyQt5
Failed to import QtCore; problem involved: C:\Users\Admin\miniconda3\envs\pybert\lib\site-packages\PyQt5\QtCore.pyd.
Failed to import PyQt6; problem involved: None.
Failed to import PyQt4; problem involved: None.
Cannot import any of PySide2, PySide6, PyQt5, PyQt6, PyQt4

@capn-freako
Copy link
Author

Hi @corranwebster , can you approve the workflows?

Thanks!
-db

@capn-freako
Copy link
Author

Hi @rkern , @EdwinWiseOne , @stefanoborini , @christianbrodbeck , @corranwebster ,

Is this still of any interest?

Thanks,
-db

@christianbrodbeck
Copy link
Contributor

Looks to me like it could be helpful for debugging in some cases.

@capn-freako
Copy link
Author

Looks to me like it could be helpful for debugging in some cases.

Thanks, Christian!
Are you able to unblock the workflows?

@corranwebster
Copy link
Contributor

Apologies - I think I missed this back in March. Thanks for the contribution.

This looks useful, assuming tests pass I will merge.

@corranwebster
Copy link
Contributor

Test failures appear to be due to incompatibilities between Qt5 and the latest Pillow version, which is orthogonal to this PR.

@capn-freako
Copy link
Author

Test failures appear to be due to incompatibilities between Qt5 and the latest Pillow version, which is orthogonal to this PR.

So, do I have an action here?

@corranwebster
Copy link
Contributor

So, do I have an action here?

If you can merge in current main, that should like resolve the issue. I can possibly do it but won't have the spare cycles for at least a week.

@capn-freako
Copy link
Author

So, do I have an action here?

If you can merge in current main, that should like resolve the issue. I can possibly do it but won't have the spare cycles for at least a week.

I'm being told I already have everything:

% git status
On branch fix-issue-1205
Your branch is up to date with 'origin/fix-issue-1205'.

nothing to commit, working tree clean

% git merge upstream/main
Already up to date.

@corranwebster
Copy link
Contributor

I'm being told I already have everything:

Thanks. Then it's on me to sort out what's going on with the errors.

@capn-freako
Copy link
Author

capn-freako commented Dec 15, 2023 via email

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