-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Should i686 and arm64 builds be failing in the wheel repo? #19147
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
Comments
Both of these are built and tested on Fedora and pass. A couple of the tests are new and the failure on aarch64 might need new tolerances. The i686 ones really shouldn't be failing so much; I think this must be a difference between compiler arguments and/or actual hardware involved in floating-point calculations. |
From what I can gather, there are the following CI runs on github actions
There is an There is a Where are the logs from the Fedora runs on HEAD? Would the following make sense?
If that plan is acceptable I could try to get some PRs going. An admin would have to sign up with a service like shipppable.com for arm64 (which is what numpy uses) |
I see I missed the azure runs which build and test on (linux, macos, windows) * (python 3.7, 3.8) and windows python version "Pre" |
Fedora does not run HEAD, just releases (and release candidates), which is why the new tests are not yet fixed. |
The last AppVeyor run was today? |
We no longer build for 32-bit, and arm64 wheels are building fine with cibuildwheel now for a while. The MacPython wheel building system is no longer used. |
Repeating this comment here in the hopes it gets greater visibility. The MacPython/matplot-lib repo builds and uploads wheels weekly, and also can provide them for uploading to PyPI. It has been broken for a while. I tried reviving it in a PR to move the x86 builds to github actions, and leave only the arm64 ones on travis.com.
Should the arm64 and i686 ones be failing? The strategy in that repo is to build in a docker image and to test in a different one, perhaps that is breaking the image tests?
The text was updated successfully, but these errors were encountered: