Skip to content

Commit 651f003

Browse files
authored
Merge pull request #1123 from oesteban/enh/reports-nitpicks
[ENH] Some nit picks of the reports
2 parents 75c2b6d + 9db1d22 commit 651f003

File tree

2 files changed

+19
-16
lines changed

2 files changed

+19
-16
lines changed

fmriprep/interfaces/reports.py

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@
4444
"""
4545

4646
ABOUT_TEMPLATE = """\t<ul>
47-
\t\t<li>FMRIPREP version: {version}</li>
48-
\t\t<li>FMRIPREP command: <tt>{command}</tt></li>
47+
\t\t<li>FMRIPrep version: {version}</li>
48+
\t\t<li>FMRIPrep command: <code>{command}</code></li>
4949
\t\t<li>Date preprocessed: {date}</li>
5050
\t</ul>
5151
</div>
@@ -106,7 +106,7 @@ def _generate_segment(self):
106106
if recon.cmdline.startswith('echo'):
107107
freesurfer_status = 'Pre-existing directory'
108108
else:
109-
freesurfer_status = 'Run by FMRIPREP'
109+
freesurfer_status = 'Run by fMRIPrep'
110110

111111
output_spaces = [self.inputs.template if space == 'template' else space
112112
for space in self.inputs.output_spaces]
@@ -127,7 +127,7 @@ def _generate_segment(self):
127127
header = '\t\t<ul class="elem-desc">'
128128
footer = '\t\t</ul>'
129129
lines = ['\t\t\t<li>Task: {task_id} ({n_runs:d} run{s})</li>'.format(
130-
task_id=task_id, n_runs=n_runs, s='' if n_runs == 1 else 's')
130+
task_id=task_id, n_runs=n_runs, s='' if n_runs == 1 else 's')
131131
for task_id, n_runs in sorted(counts.items())]
132132
tasks = '\n'.join([header] + lines + [footer])
133133

@@ -164,13 +164,16 @@ def _generate_segment(self):
164164
stc = {True: 'Applied',
165165
False: 'Not applied',
166166
'TooShort': 'Skipped (too few volumes)'}[self.inputs.slice_timing]
167-
reg = {'FSL': [
168-
'FLIRT with boundary-based registration (BBR) metric - %d dof' % dof,
169-
'FLIRT rigid registration - 6 dof'],
170-
'FreeSurfer': [
171-
'FreeSurfer boundary-based registration (bbregister) - %d dof' % dof,
172-
'FreeSurfer mri_coreg - %d dof' % dof],
173-
}[self.inputs.registration][self.inputs.fallback]
167+
reg = {
168+
'FSL': [
169+
'FSL <code>flirt</code> with boundary-based registration'
170+
' (BBR) metric - %d dof' % dof,
171+
'FSL <code>flirt</code> rigid registration - 6 dof'],
172+
'FreeSurfer': [
173+
'FreeSurfer <code>bbregister</code> '
174+
'(boundary-based registration, BBR) - %d dof' % dof,
175+
'FreeSurfer <code>mri_coreg</code> - %d dof' % dof],
176+
}[self.inputs.registration][self.inputs.fallback]
174177
if self.inputs.pe_direction is None:
175178
pedir = 'MISSING - Assuming Anterior-Posterior'
176179
else:

fmriprep/viz/config.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -102,25 +102,25 @@
102102
"name": "epi_mean_t1_registration/flirt",
103103
"file_pattern": "func/.*_bold_flirt",
104104
"title": "EPI to T1 registration",
105-
"description": "FLIRT was used to generate transformations from EPI space to T1 Space - BBR refinement rejected"
105+
"description": "FSL <code>flirt</code> was used to generate transformations from EPI space to T1 Space - BBR refinement rejected"
106106
},
107107
{
108108
"name": "epi_mean_t1_registration/coreg",
109109
"file_pattern": "func/.*_bold_coreg",
110110
"title": "EPI to T1 registration",
111-
"description": "mri_coreg (FreeSurfer) was used to generate transformations from EPI space to T1 Space - bbregister refinement rejected"
111+
"description": "<code>mri_coreg</code> (FreeSurfer) was used to generate transformations from EPI space to T1 Space - <code>bbregister</code> refinement rejected"
112112
},
113113
{
114114
"name": "epi_mean_t1_registration/flt_bbr",
115115
"file_pattern": "func/.*_bold_flt_bbr",
116116
"title": "EPI to T1 registration",
117-
"description": "FLIRT was used to generate transformations from EPI space to T1 Space - FAST segmentation used for BBR"
117+
"description": "FSL <code>flirt</code> was used to generate transformations from EPI-space to T1w-space - The white matter mask calculated with FSL <code>fast</code> (brain tissue segmentation) was used for BBR"
118118
},
119119
{
120120
"name": "epi_mean_t1_registration/bbr",
121121
"file_pattern": "func/.*_bold_bbr",
122122
"title": "EPI to T1 registration",
123-
"description": "bbregister was used to generate transformations from EPI space to T1 Space"
123+
"description": "<code>bbregister</code> was used to generate transformations from EPI-space to T1w-space"
124124
},
125125
{
126126
"name": "epi/carpetplot",
@@ -132,7 +132,7 @@
132132
"name": "ica_aroma",
133133
"file_pattern": "func/.*ica_aroma",
134134
"title": "ICA AROMA",
135-
"description": "Maps created with maximum intensity projection (glass brain) with a black brain outline. Right hand side of each map: time series (top in seconds), frequency spectrum (bottom in Hertz). Components classified as signal in green; noise in red."
135+
"description": "Maps created with maximum intensity projection (glass brain) with a black brain outline. Right hand side of each map: time series (top in seconds), frequency spectrum (bottom in Hertz). Components classified as signal are plotted in green; noise components in red."
136136
}
137137
]
138138
},

0 commit comments

Comments
 (0)