example app now works with Qt4 and Python2 again#1302
example app now works with Qt4 and Python2 again#1302j9ac9k merged 11 commits intopyqtgraph:masterfrom
Conversation
|
Is there a reason our tests didn't catch this? We do run the examples in the test suite, pretty sure. |
|
Our test site runs the examples, but not the example application funny enough. Should I look to add launching the example application as part of the test suite? Also that CI failure, I am able to intermittently reproduce locally, but not reliably. |
|
Been thinking, I think using |
|
Hmm getting the example app to run as part of the test suite is tricky haha |
98ea370 to
9dd5372
Compare
|
@ksunden can you think of a way we can add the example app as part of the test suite? I just added it to https://github.com/pyqtgraph/pyqtgraph/blob/master/examples/utils.py#L93 where instead of |
|
whoops, accidentally pushed the wrong commit; going to put this in draft commit form until ready... |
51d0f59 to
8634834
Compare
8634834 to
fff6567
Compare
159f828 to
18b015a
Compare
|
While this branch now works for having the example application launch on Python2 successfully, I want to try and minimize the warnings on python3 when launching the application, specifically.... I get this warning when launching examples via import pyqtgraph.examples
pyqtgraph.examples.run()or $ python examplesEDIT: lastly, when installing pyqtgraph in editable mode, running |
|
Scary warning no longer comes up; still want |
|
Apparently running examples in editable mode is related to a long-standing setuptools issue: pypa/setuptools#230 We can work around this by changing the directory structure, but I don't think we actually want to do that for the additional functionality... last issue is fixing the font warning... |
8a51295 to
c4ad4ae
Compare
|
Switched font family to Courier New, which is available everywhere by default, had to regenerate the uic templates (always annoying)... Did last bit of code cleanup, I think this PR is ready for merging. |
|
just kidding, still not ready for merging, test app didn't actually get run... |
|
ok, now the test application is tested as part of the test suite. |
6d91dd0 to
05ee86c
Compare
|
ok now that docs/CI and so on work, I think I'll just merge this guy... |
This PR fixes an issue introduced in #1124 where the example application was not compatible with Python2 (due to a python3 method of calling
super()) and Qt4 (due to depending onpaletteChangedsignal which only exists in Qt5).Fixes #1301