NSO-RPC icon indicating copy to clipboard operation
NSO-RPC copied to clipboard

Upgrade to pyqt6

Open DarthMDev opened this issue 2 years ago • 3 comments

Is there any particular reason we are staying on pyqt5?

DarthMDev avatar Sep 14 '22 04:09 DarthMDev

I think I initially had some issues with using pyqt6 on my M1 Mac, so I just defaulted back to pyqt5 which has worked for me in the past. I'll look into updating this soon, if you'd like.

MCMi460 avatar Sep 14 '22 05:09 MCMi460

Ah ok if I were to try to contribute and fork what would be a good replacement for qtwidgets

DarthMDev avatar Sep 14 '22 05:09 DarthMDev

I can't find any myself, but there appears to be a PR on the qtwidgets Github page (pythonguis/python-qtwidgets#5) which may work as an alternative. However, it has been over a year and a half since the repository has last been updated, I wouldn't get my hopes up that the PR would be merged anytime soon. Instead of qtwidgets, you could just change it to be a QPushButton that changes color upon being clicked, or just try to implement your own version of a similar toggle switch. I'm not sure exactly what the advantages of using PyQt6 over PyQt5 is, but if you'd really like, I can attempt to port it over whenever I have free time next week.

MCMi460 avatar Sep 15 '22 15:09 MCMi460

Since this seems nigh impossible (for the lazy such as I), I will avoid doing this as soon as possible. Thank you for the suggestion, though. If PyQt5 becomes outdated enough to begin causing problems for some users (or PyInstaller breaks it), then I will consider updating to PyQt6 and re-implementing qtwidgets of my own accord, but until then, this will probably not come to fruition.

MCMi460 avatar May 26 '23 05:05 MCMi460