-
Notifications
You must be signed in to change notification settings - Fork 132
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
IndexError in the initial steps of the participant level when saving the config file #1264
Comments
Okay let's dissect this:
Before replicating myself, can I ask you for a last piece of information? Can you please copy and paste the contents of your config file? The file should be named something like: |
Thanks, @oesteban -- I'm also confused by the version number that is MRIQC outputs, and I've noticed that discrepancy for a few versions now. I'm not sure what it means for the environment to be leaking. For reference, here's how I built the container:
Also, here's the contents of the config file in scratch directory:
|
I suspect this comes from mixing multi-echo and single-echo inputs - I'll check tomorrow and submit a fix. |
Thanks, Oscar! That sounds like a possibility given where it seems to break in these data (e.g., sub-10015 doesn't have multiecho and MRIQC works). |
Confirmed - the problem is not because some runs are multi-echo. The problem comes from having both (single-echo and multi-echo) in a single run. |
This happened, e.g., when trying to run MRIQC on datasets with both single-echo and multi-echo runs or with filters where some array had mixed types (e.g., `part: [null, 'mag']`). Resolves: #1264.
(forked from #1087)
First post
Originally posted by @DVSneuro in #1087 (comment)
I'm getting the same IndexError with the latest pre-release for 24.0.0, but maybe it's a conflicting environment issue? I'm using
--cleanenv
so I thought that option would prevent that, but I am likely misunderstanding something.Using the same data and code, I'm not getting any errors with 23.1.0. Maybe one notable feature of my dataset is that it is multiecho, and it seems like the handling for multiecho data has improved since 23.1.0. The IndexError appears starting with 23.1.1.
Happy to share data if it's helpful. It's not quite ready for OpenNeuro and public sharing, but maybe I send a few subjects privately via OneDrive or DropBox.
Thanks for any help!
Best wishes,
David
Here is my code:
Here is the contents of my mriqc_config.json file:
And here is the full output of the error:
Follow up post
Originally posted by @DVSneuro in #1087 (comment)
Thanks, @oesteban -- sorry, I wasn't sure if I should create a new issue for this error. I do get the same error with the 24.0.0rc2 version. I can also confirm that it happens without the config file being used.
I got a different error with a different subject (see below). So, just as a sanity check, I tripled checked the bids validator, and there are no errors there.
Same error, latest pre-release:
Different subject, new error:
Data
@DVSneuro posted a MWE on openneuro: https://openneuro.org/datasets/ds005085
The text was updated successfully, but these errors were encountered: