-
Notifications
You must be signed in to change notification settings - Fork 2
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
Test failures with test_transform on NixOS #20
Comments
Thanks for the feedback. |
In the meantime, I have fixed a couple of bugs with latest PyQt6 and PySide6 versions and Python 3.12 on Linux. |
I'm using PyQt5, and Numpy 1.26 . With PyQt6, PySide6 and even PySide2 I experience #11 - very similar to PlotPyStack/guidata#77 - which was indeed fixed (thanks). I hoped that including the patch that fixed PlotPyStack/guidata#77 would fix #11 for me, but it doesn't. Here are more logs of the With PySide2
With PyQt6
With PySide6
With PyQt5, I don't experience #11 , but I do experience the error from the first comment. Here's how pytest starts:
|
Oh and With guidata 3.6.3, I get a totally different error:
Which made me try out the latest with PySide2
with PyQt5
I strongly suspect the |
Attempt to fix Test failures with test_transform on NixOS #20
Sorry for not getting back to you sooner - I was quite busy these days. PyQt5 PySide2 |
May contribute fixing Test failures with test_transform on NixOS #20
Thanks for that fix. It is confirmed. I didn't fully delve into the details there, but I suspect it is because we use Numpy 1.26, and not Numpy 2.x, with which you probably are used to run the tests. Don't ask why we are tied to using Numpy 1.x 😅.
With NixOS it is easy to test all of them :). I will only layout here which ones worked:
I guess it is very much OK. PySide2 and PyQt5 shouldn't be used for new, modern applications. It'd be nice to document that PySide2 is not supported, and hence it'd be "officially" not supported. |
Great! 🥲 I won't, the thread on discourse in pretty dissuasive 🙂.
OK, so we are on the same wavelength. 👍
That is now documented in the Thanks for your work! |
That looks great, thanks for that :).
Thank you! |
As part of my packaging effort, here are 2 small failures we experience in NixOS in NixOS/nixpkgs#327446 :
The text was updated successfully, but these errors were encountered: