-
Notifications
You must be signed in to change notification settings - Fork 8
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
MacOS ARM should use PyQt5 backend instead of PySide2 #35
Comments
Reposting the important bit:
Interestingly the vanilla conda solver gives pyqt:
Edit: as seen here: napari/napari#4891 (comment) just installing napari from conda-forge does give the pyqt backend. Not sure why the stardist-napari makes it pyside2. |
Bumping this. The root of the problem seems to be that
This should print True but will print False on arm64. I checked the x86 version in Rosetta is fine. |
I assume there are no I am fine using I'll check if this is available in the |
I wonder if there are mitigations we can do from the napari side (monkey patching or worse!), though. |
Oh, good ideas! |
Well, good news, it's not conda-forge's fault :) $ conda deactivate # do not use conda envs
$ brew install pyside@2
$ brew list pyside@2 # write down location of site-packages
$ python3 # this should be brew's python, check with `which pyhrton3`
Python 3.10.8 (main, Oct 13 2022, 09:48:54) [Clang 13.0.0 (clang-1300.0.29.30)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.path.append("/opt/homebrew/Cellar/pyside@2/5.15.5_3/lib/python3.10/site-packages/")
>>> import PySide2
>>> from PySide2.QtWidgets import QOpenGLWidget
>>> hasattr(QOpenGLWidget, "defaultFramebufferObject")
False So... oopsie, I don't think we can do much at the conda-forge level without a patch. |
Well then, I guess it's an upstream issue related to building on arm64? |
Probably best reported at https://wiki.qt.io/Qt_for_Python/Reporting_Bugs, I assume. I couldn't find any issues about this particular problem there. I guess it would be a good idea to provide both conda-forge's recipe and brew's formula to help diagnose potential build configuration issues (and not strictly in the source): |
Update: it's been reported up-stream by Julien Schueller |
Awesome. Looks like it'll be fixed with 5.15.10. |
conda-forge/pyside2-feedstock#175
|
I'm happy where this thread went because "if we ignore the licensing issues" was alarming 😂 |
Oh wow! Then no reason to keep this open, right? :D |
Thanks for all the debugging and bug chasing @psobolewskiPhD! Closing here 🚀 |
So with this, can we reverse napari/napari#5255 and have arm64 bundles back? |
I think it got reverted at some point already. Current |
so it means that the last build does not update it. |
Because of a bug reported by @psobolewskiPhD, the napari fails to start if there is PySide2 backend on macOS ARM.
More details here: napari/napari#5231
The text was updated successfully, but these errors were encountered: