Skip to content

Commit

Permalink
Keep fixing.
Browse files Browse the repository at this point in the history
  • Loading branch information
tsalo committed May 7, 2024
1 parent 262734f commit 5558de3
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 12 deletions.
12 changes: 6 additions & 6 deletions src/fmripost_aroma/utils/features.py
Original file line number Diff line number Diff line change
Expand Up @@ -231,12 +231,12 @@ def feature_spatial(component_maps):
'LongName': 'CSF content fraction',
'Description': (
"The fraction of thresholded component z-values in the brain's cerebrospinal "
"fluid. "
"This is calculated by "
"(1) taking the absolute value of the thresholded Z map for each component, "
"(2) summing z-statistics from the whole brain, "
"(3) summing z-statistics from voxels in CSF compartments, and "
"(4) dividing the CSF z-statistic sum by the whole brain z-statistic sum."
'fluid. '
'This is calculated by '
'(1) taking the absolute value of the thresholded Z map for each component, '
'(2) summing z-statistics from the whole brain, '
'(3) summing z-statistics from voxels in CSF compartments, and '
'(4) dividing the CSF z-statistic sum by the whole brain z-statistic sum.'
),
'Units': 'arbitrary',
},
Expand Down
8 changes: 2 additions & 6 deletions src/fmripost_aroma/utils/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -289,9 +289,7 @@ def motpars_spm2fsl(motpars):
raise ValueError('Input must be an existing file or a numpy array.')

Check warning on line 289 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L289

Added line #L289 was not covered by tests

if motpars.shape[1] != 6:
raise ValueError(
f'Motion parameters must have exactly 6 columns, not {motpars.shape[1]}.'
)
raise ValueError(f'Motion parameters must have exactly 6 columns, not {motpars.shape[1]}.')

Check warning on line 292 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L292

Added line #L292 was not covered by tests

# Split translations from rotations
trans, rot = motpars[:, :3], motpars[:, 3:]

Check warning on line 295 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L295

Added line #L295 was not covered by tests
Expand Down Expand Up @@ -325,9 +323,7 @@ def motpars_afni2fsl(motpars):
raise ValueError('Input must be an existing file or a numpy array.')

Check warning on line 323 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L323

Added line #L323 was not covered by tests

if motpars.shape[1] != 6:
raise ValueError(
f'Motion parameters must have exactly 6 columns, not {motpars.shape[1]}.'
)
raise ValueError(f'Motion parameters must have exactly 6 columns, not {motpars.shape[1]}.')

Check warning on line 326 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L326

Added line #L326 was not covered by tests

# Split translations from rotations
trans, rot = motpars[:, :3], motpars[:, 3:]

Check warning on line 329 in src/fmripost_aroma/utils/utils.py

View check run for this annotation

Codecov / codecov/patch

src/fmripost_aroma/utils/utils.py#L329

Added line #L329 was not covered by tests
Expand Down

0 comments on commit 5558de3

Please sign in to comment.