-
Notifications
You must be signed in to change notification settings - Fork 616
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
Drop PyQt4 and PySide support #1788
Comments
This is difficult for me. We don't really have numbers on how many people are using Qt4. Since it was basically the preferred backend for vispy (most of the examples were Qt based if they had to be and Qt is the first backend chosen if there are multiple) for a long time. Switching to |
I understand, I feel like both Python2 -> Python3 and Qt4->Qt5 migrations were REALLY hard. That said, they both happened. PyQt4 isn't even released anymore on PyPi The Users would be able to do
and The main advantage is that all the backward compatibility logic is handled by We also get to leverage all the hard work that Spyder is doing. |
Assuming that The environment variable stuff should never be something a vispy user has to deal with. We can hack that in to our existing app import stuff if needed. |
I agree the environment variable stuff shouldn't be used by a typical user. |
we've been using I actually wrote a |
I think it would be best to do our next release (0.7) and then remove Qt4. If someone wants to create a pull request to do this, please do and let me know that you're working on it. |
Would you accept a switch to qtpy? Do you anticipate conflicts on this if somebody starts working on it now before the 0.7 release? |
Qtpy doesn't seem to support PyQt6 (spyder-ide/qtpy#225) which was just added to VisPy. I still don't really like the extra dependency, but let's go for it if it simplifies things. No problem getting a PR before 0.7 release as I won't merge it until after and I don't expect any changes to the Qt backend. |
Understood. |
I'm adding this to the PyCascades 2022 sprint project. I think as an initial point of completion we should remove all code in the |
I think PyQt4 should be dropped.
It would really simplify LOTs of the code.
As an addendum, I think
qtpy
should probably be used as the only qt "backend", and people should be pointed to it in selecting their backend: https://github.com/spyder-ide/qtpyTo add to confusion, there exists a competitor,
Qt.py
https://github.com/mottosso/Qt.pyThe text was updated successfully, but these errors were encountered: