[ENH] Derive masks to support native-space data#24
[ENH] Derive masks to support native-space data#24tsalo wants to merge 7 commits intoME-ICA:mainfrom
Conversation
|
Thank you @tsalo ! I tried running the integration tests and it fails. I think it's related to |
|
I didn't get any errors in the workflow itself, but it looks like the masks are different enough that the ICA produces different components, which is causing the test to fail. |
You guys mean locally right? Because the integration tests on Circle don't seem to have ever worked, correct? |
|
Right. The integration tests are going to fail on CircleCI until we get rid of FSL completely- unless we set up a Dockerfile with FSL installed, which we haven't done because we ultimately want to drop the dependency anyway. |
Co-authored-by: Oscar Esteban <code@oscaresteban.es>
|
That's a weird error you got here. Is it somehow related to nilearn's |
References #2 and stems from Brainhack-Donostia/ica-aroma-org#49.
Changes proposed in this pull request: