manim icon indicating copy to clipboard operation
manim copied to clipboard

Test on Apple Silicon ARM64

Open cclauss opened this issue 1 month ago • 3 comments

Overview: What does this pull request change?

Test on Apple Silicon ARM64

Motivation and Explanation: Why and how do your changes improve the library?

Ensure that our CI tests pass on modern Macs.

  • https://github.com/actions/runner-images

The currently available GitHub Actions macOS runners are:

macOS Version runner.arch
macos-14 ARM64
macos-15 ARM64
macos-15-intel X64
macos-26 ARM64
macos-latest ARM64

Links to added or changed documentation pages

Further Information and Comments

Reviewer Checklist

  • [ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • [ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • [ ] If applicable: newly added functions and classes are tested

cclauss avatar Dec 01 '25 13:12 cclauss

I wonder if we should drop testing for macos-15-intel altogether and only use macos-latest -- AFAIR there were some build / linker issues with cairo when @chopan050 tried that recently. Does the combination of macos-latest with Python 3.13 just play nice(r)?

behackl avatar Dec 02 '25 00:12 behackl

I think those Cairo issues are more about the macOS cache logic at https://github.com/ManimCommunity/manim/blob/a50d8363e75f748f0c6462267d4829e38fefa41e/.github/workflows/ci.yml#L77

Perhaps it also needs to capture runner.arch so that Intel and ARM caches do not overwrite each other.

cclauss avatar Dec 02 '25 03:12 cclauss

OK! macos-latest passes on 4 Pythons. macos-15-intel running on Py3.13 (because it is Required).

cclauss avatar Dec 02 '25 04:12 cclauss