Skip to content

Feature/macos arm#23

Merged
olivhoenen merged 20 commits intoiterorganization:developfrom
paulotex:feature/macos-arm
Feb 9, 2026
Merged

Feature/macos arm#23
olivhoenen merged 20 commits intoiterorganization:developfrom
paulotex:feature/macos-arm

Conversation

@paulotex
Copy link
Contributor

@paulotex paulotex commented Jan 16, 2026

Improving the macOS ARM version:

  • Works for python 3.10 - 3.13, like the other architectures.
  • Includes UDA backend.

paulotex and others added 13 commits January 8, 2026 10:49
@paulotex
Adding config for macos, still untested.
5e06e59
Fixing RPATH issues
a1e0d32
@paulotex
Cleaning main CMakeLists. Seems to work on macOS now.
c96598b
@paulotex
ensuring arm64 when building the wheel on macos
5d51fa9
@paulotex
Removing github actions for macOS x86 for now, will be added later on…
38d42e1 
… request.
@paulotex
Fixing macOS version from 11 to 14 to ensure arm only, no x86
c63d4f4
@paulotex
Improving legibility in macOS section in cmake
1ece600
@paulotex
Fixing missing override warnings
b44579c
@paulotex
Merge branch 'develop' into feature/macos-arm
415d503
@paulotex
Adding support for python versions 3.10 to 3.13, like the other archi…
a34ef25 
…tectures.
@paulotex
Adding missing override modifiers to avoid compilation warnings
c6ce7a5
@paulotex
macOS ARM: all python versions available and UDA enabled.
72dffdd
@paulotex
Adding missing override to virtual functions implementations.
c9e92a1
@paulotex paulotex marked this pull request as draft January 16, 2026 12:45
@paulotex
Copy link
Contributor Author

paulotex commented Jan 16, 2026

UDA build is failing, seems to target an old macOS version. Checking...

@olivhoenen olivhoenen mentioned this pull request Jan 21, 2026
Merged
@paulotex
Copy link
Contributor Author

paulotex commented Jan 21, 2026

The issue here is that capnp is not standalone, it uses libs that are installed by brew. I have to change the CI /cmake to ensure that capnp is standalone so the the wheel can include it without pointing to libs outside the wheel.

@olivhoenen
Copy link
Collaborator

olivhoenen commented Jan 22, 2026

@paulotex can we focus this PR on expanding the matrix for MacOS up to Python 3.14 and verify that the wheel works well when there is no local brew install of hdf5? @adam-parker1 may be helping testing the wheel

We can add UDA support in a subsequent PR

@olivhoenen olivhoenen added this to the release 5.6.0 milestone Jan 22, 2026
olivhoenen
olivhoenen reviewed Jan 28, 2026
View reviewed changes
@paulotex paulotex marked this pull request as ready for review February 6, 2026 16:28
@olivhoenen
Copy link
Collaborator

olivhoenen commented Feb 8, 2026

https://github.com/iterorganization/IMAS-Core/actions/runs/21757616815/job/62772211683?pr=23#step:7:1255 it seems to pick HDF5 2.0 instead of 1.14 in previous passing build. Could you check @prasad-sawantdesai?

@olivhoenen olivhoenen merged commit 2826a07 into iterorganization:develop Feb 9, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@olivhoenen olivhoenen olivhoenen approved these changes

@SimonPinches SimonPinches Awaiting requested review from SimonPinches SimonPinches is a code owner

@prasad-sawantdesai prasad-sawantdesai Awaiting requested review from prasad-sawantdesai prasad-sawantdesai is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

release 5.6.0

Development

Successfully merging this pull request may close these issues.

3 participants

@paulotex @olivhoenen @prasad-sawantdesai