Skip to content

Add support for running pyside2-uic binary to dynamically compile ui files#1211

Merged
campagnola merged 8 commits intopyqtgraph:developfrom
campagnola:pyside2-uic
Jun 4, 2020
Merged

Add support for running pyside2-uic binary to dynamically compile ui files#1211
campagnola merged 8 commits intopyqtgraph:developfrom
campagnola:pyside2-uic

Conversation

@campagnola
Copy link
Copy Markdown
Member

fixes #1102

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 28, 2020

Hmm, all linux CI runs are failing, but not just on this PR, but on another one as well... wonder if there is some upstream issue.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 29, 2020

Closing/Opening to re-trigger CI run

@j9ac9k j9ac9k closed this May 29, 2020
@j9ac9k j9ac9k reopened this May 29, 2020
@2xB
Copy link
Copy Markdown
Contributor

2xB commented May 29, 2020

#1102 is closed, does that affect this PR?

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 29, 2020

@2xB that was my fault, I didn't verify the solution worked entirely and closed the issue, turns out we do need a change in the pyqtgraph code-base

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Jun 4, 2020

I think this branch is ready to merge @campagnola ... figured we should at least test the new loadUiType functionality in the CI platform, so I added a pyside2-latest pipeline (now up to 18, yay!)

@campagnola campagnola merged commit 932b975 into pyqtgraph:develop Jun 4, 2020
@campagnola campagnola deleted the pyside2-uic branch June 4, 2020 08:07
@campagnola
Copy link
Copy Markdown
Member Author

Thanks @j9ac9k !

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.

pyqtgraph.Qt.loadUiType broken for PySide2 >= 5.14

3 participants