[BUG]: Change type "confound" to "BOLD_confound" in preprocessing for consistency with data grabber #81

New issue

Closed

opened 2022-09-15 09:13:53 +00:00 by verakye · 1 comment

verakye commented

2022-09-15 09:13:53 +00:00

(Migrated from github.com)

Is there an existing issue for this?

I have searched the existing issues

Current Behavior

The types naming in the preprocessing and the data grabbing has to be consistent. In the data grabbing the "desc-confounds_regressors" from fmriprep are currently names as BOLD_confounds but in the preprocessing this is referred to as confounds.

Expected Behavior

The above described naming has to be adapted in the preprocessing for consistency.

Steps To Reproduce

Environment

Environment as specified in the repos `condo-env.yml`.

Relevant log output

No response

Anything else?

No response

### Is there an existing issue for this? - [X] I have searched the existing issues ### Current Behavior The `types` naming in the preprocessing and the data grabbing has to be consistent. In the data grabbing the "desc-confounds_regressors" from fmriprep are currently names as `BOLD_confounds` but in the preprocessing this is referred to as `confounds`. ### Expected Behavior The above described naming has to be adapted in the preprocessing for consistency. ### Steps To Reproduce - ### Environment ```markdown Environment as specified in the repos `condo-env.yml`. ``` ### Relevant log output _No response_ ### Anything else? _No response_

fraimondo commented

2022-11-02 12:24:42 +00:00

(Migrated from github.com)

addressed in #111