Skip to content

Fix cylinder intersection failures on ARM64, test for almost equality#671

Merged
MridulS merged 3 commits intomainfrom
fma_arm
Feb 12, 2026
Merged

Fix cylinder intersection failures on ARM64, test for almost equality#671
MridulS merged 3 commits intomainfrom
fma_arm

Conversation

@MridulS
Copy link
Member

@MridulS MridulS commented Feb 11, 2026

fixes #669

@MridulS
Copy link
Member Author

MridulS commented Feb 11, 2026

- name: Set up MATLAB
uses: matlab-actions/setup-matlab@v2
with:
release: R2023b
Copy link
Member Author

Choose a reason for hiding this comment

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

This gets the latest MATLAB runtime for github actions which has fixed some issues with ARM machines.

Copy link
Member

Choose a reason for hiding this comment

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

Ok. The problem is that horace does not officially support the latest matlab. It seems to work but we can't necessarily rely on that.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, we put in a pin when it fails?

Copy link
Member

Choose a reason for hiding this comment

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

Ok

@nvaytet
Copy link
Member

nvaytet commented Feb 11, 2026

The macos tests are fixed: https://github.com/scipp/scippneutron/actions/runs/21905945879?pr=671

The osx tests passed, but the windows tests failed...

@jl-wynen
Copy link
Member

The macos tests are fixed: https://github.com/scipp/scippneutron/actions/runs/21905945879?pr=671

The osx tests passed, but the windows tests failed...

Is that still the case? I don't see the failed test

@MridulS
Copy link
Member Author

MridulS commented Feb 12, 2026

Yeah I rerun the test, it was a flaky hypothesis test timing out.

@nvaytet
Copy link
Member

nvaytet commented Feb 12, 2026

Yeah I rerun the test, it was a flaky hypothesis test timing out.

Maybe we should address that? Or at least open an issue to fix it in a follow-up?

@MridulS
Copy link
Member Author

MridulS commented Feb 12, 2026

created #672

@MridulS MridulS merged commit f9a54b1 into main Feb 12, 2026
5 checks passed
@MridulS MridulS deleted the fma_arm branch February 12, 2026 14:30
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.

Weekly osx builds fail

3 participants