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
Failures of some tests for load_confounds on mac OS 14 #4404
Comments
Last successful run on main https://github.com/nilearn/nilearn/actions/runs/8831115070/job/24245630266#step:1:11
|
seems that this is OS version related |
We may not be the only ones facing this... |
Yes there are times in my life when I want to feel special and I seriously hope this is not one those. |
Confirmed here that previous mac OS versions did not fail: |
Until this gets resolved I am tempted to change CI so to test on both macos 13 and 14 so make it clear on seeing tests results that this an OS issue. |
Indeed |
Testing things in a separate repo to iterate more quickly: https://github.com/Remi-Gau/nilearn_tmp As far as I can tell the tests only fail when using detrend when masking the data with confounds. So far I have not seen anyone raise an issue about this but I think that my google / github fu is failing me. @htwangtw Ideally I think we'd want to have a very reduced example demonstrating the issue, so we can report it in the correct place. |
I feel like I have been putting this off for too long... In this set of tests I was just checking if the denoising gets some value below an arbitrary range, therefore numerical instability can really tip things off... I will have to refresh my memory and think of a better set of tests |
I am going to work on this today and tomorrow |
With some help from @anibalsolon, we noticed that it was all the test with |
See: https://github.com/nilearn/nilearn/actions/runs/8831115070
https://github.com/nilearn/nilearn/actions/runs/8831115070/job/24263341338#step:6:16805
OS and dependencies details
The text was updated successfully, but these errors were encountered: