Robustify fMRIprep hack #4

Merged

adswa merged 1 commit from robustify-fmriprep into main 2022-05-04 15:19:53 +00:00

Conversation 2 Commits 1 Files changed 1 +2 -2

adswa commented 2021-11-22 07:14:34 +00:00 (Migrated from github.com)

Copy link

To work araound a bug in fMRIprep, we move all json files in
a non-json-extension (else old version of fmriprep will attempt
to read them and fail if they aren't retrieved). @jkroell encountered
a BIDS-ish dataset without any jsons, where the find...mv
command failed. The --no-run-if-empty option of xargs safeguards
this, preventing mv to be run if the find command does not return
any results

To work araound a bug in fMRIprep, we move all json files in a non-json-extension (else old version of fmriprep will attempt to read them and fail if they aren't retrieved). @jkroell encountered a BIDS-ish dataset without any jsons, where the ``find...mv`` command failed. The ``--no-run-if-empty`` option of ``xargs`` safeguards this, preventing mv to be run if the find command does not return any results

adswa commented 2022-05-04 12:30:27 +00:00 (Migrated from github.com)

Copy link

@mih would you mind reviewing this?

@mih would you mind reviewing this?

mih (Migrated from github.com) approved these changes 2022-05-04 15:14:33 +00:00

mih (Migrated from github.com) left a comment

Copy link

LGTM. Thx!

LGTM. Thx!

Sign in to join this conversation.

Reviewers

No reviewers

mih

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

datalad/fairly-big-processing-workflow!4

No description provided.