Skip to content

ci: Add an ARM Linux test workflow #29477

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

Merged
merged 1 commit into from
Jan 18, 2025
Merged

ci: Add an ARM Linux test workflow #29477

merged 1 commit into from
Jan 18, 2025

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Jan 17, 2025

PR summary

Like the macOS jobs are mixed between Intel and ARM, this changes one of the Linux jobs to use ARM.

We'll see if we need to make any changes as in #24597 after the build is complete.

PR checklist

@QuLogic QuLogic added the CI: testing CI configuration and testing label Jan 17, 2025
@QuLogic
Copy link
Member Author

QuLogic commented Jan 17, 2025

I should probably skip some backends though; it seems PyQt5 doesnt have a wheel and is just stuck.

echo 'PyQt5 is not available'
# PyQt5 does not have any wheels for ARM on Linux.
if [[ "${{ matrix.os }}" != 'ubuntu-20.04-arm' ]]; then
python -mpip install --upgrade pyqt5${{ matrix.pyqt5-ver }} &&
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
python -mpip install --upgrade pyqt5${{ matrix.pyqt5-ver }} &&
python -mpip install --upgrade pyqt5${{ matrix.pyqt5-ver }} --only-binary pyqt5 &&

The latest CI job seems like it is trying to build pyqt5 from an sdist still.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I messed up the condition, but I've added this as well.

@tacaswell tacaswell added this to the v3.11.0 milestone Jan 17, 2025
@QuLogic
Copy link
Member Author

QuLogic commented Jan 17, 2025

There are two failures:

  • test_preset_clip_paths, which we already add a tolerance on arm64 (aka Apple Silicon), so I've added aarch64 there as well.
  • test_pcolormesh_alpha, which we don't have any non-zero tolerance on. The diff for this one is:
    pcolormesh_alpha-failed-diff
    I don't really know why this one is different, but it seems small enough to just increase.

Like the macOS jobs are mixed between Intel and ARM, this changes one of
the Linux jobs to use ARM.
@timhoffm timhoffm merged commit 3fb9c09 into matplotlib:main Jan 18, 2025
39 checks passed
@QuLogic QuLogic deleted the arm-ci branch January 18, 2025 02:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: testing CI configuration and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants