Skip to content
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

Add CI rules for macOS. #512

Merged
merged 1 commit into from
Aug 3, 2024
Merged

Add CI rules for macOS. #512

merged 1 commit into from
Aug 3, 2024

Conversation

mmuetzel
Copy link
Contributor

@mmuetzel mmuetzel commented Aug 2, 2024

Use packages from Homebrew as dependencies.

The GitHub-hosted runners with macOS 13 are on Intel hardware. The ones with macOS 14 are on Apple silicon. Add runners for both (with otherwise the same configuration).

Quite a few tests are failing. I haven't looked into the reason why. (And I don't have physical access to a Mac.)

Use packages from Homebrew as dependencies.

The GitHub-hosted runners with macOS 13 are on Intel hardware. The ones
with macOS 14 are on Apple silicon. Add runners for both (with otherwise
the same configuration).
@raback
Copy link
Contributor

raback commented Aug 3, 2024

Great! I think we can gradually debug these various platforms this way too. Just takes some time to narrow down the issues.

@raback raback merged commit 30c4a84 into ElmerCSC:devel Aug 3, 2024
5 of 8 checks passed
@raback
Copy link
Contributor

raback commented Aug 3, 2024

Looking at the 9 failing tests in MacOS all seem to have complex valued linear systems.

@raback
Copy link
Contributor

raback commented Aug 5, 2024

My colleague had a faint recollection that these failing tests where related to the BLAS routines of MacOS. Using OpenBLAS fixed the issue according to his recollection.

@mmuetzel
Copy link
Contributor Author

mmuetzel commented Aug 5, 2024

My colleague had a faint recollection that these failing tests where related to the BLAS routines of MacOS. Using OpenBLAS fixed the issue according to his recollection.

Let's give that a try: #515

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants