Apply suggestions from code review

Co-authored-by: Oscar Esteban <code@oscaresteban.es>

fmriprep/cli/parser.py

@@ -403,11 +403,12 @@ def _slice_time_ref(value, parser):
     )
     g_outputs.add_argument(
         "--aggregate-session-reports",
+        dest="aggr_ses_reports",
         action="store",
         type=PositiveInt,
         default=4,
         help="Maximum number of sessions aggregated in one subject's visual report. "
-        "If exceeded, visual reports are separated per session.",
+        "If exceeded, visual reports are split by session.",
     )
     g_outputs.add_argument(
         "--medial-surface-nan",

fmriprep/cli/run.py

@@ -212,11 +212,7 @@ def main():
         from fmriprep.reports.core import generate_reports
 
         # Generate reports phase
-        session_list = (
-            config.execution.bids_filters["bold"]["session"]
-            if config.execution.bids_filters
-            else None
-        )
+        session_list = config.execution.bids_filters.get("bold", {}).get("session", None)
 
         failed_reports = generate_reports(
             config.execution.participant_label,

fmriprep/data/reports-spec-anat.yml

@@ -29,7 +29,7 @@ sections:
   reportlets:
   - bids: {datatype: figures, desc: about, suffix: T1w}
   - custom: boilerplate
-    path: '{reportlets_dir}/logs'
+    path: '{output_dir}/logs'
     bibfile: ['fmriprep', 'data/boilerplate.bib']
     caption: |
       <p>We kindly ask to report results preprocessed with this tool using the following boilerplate.</p>
@@ -42,6 +42,6 @@ sections:
       </p>
     title: Methods
   - custom: errors
-    path: '{reportlets_dir}/sub-{subject}/log/{run_uuid}'
+    path: '{output_dir}/sub-{subject}/log/{run_uuid}'
     captions: <em>NiReports</em> may have recorded failure conditions.
     title: Errors

fmriprep/data/reports-spec.yml

@@ -180,7 +180,7 @@ sections:
   reportlets:
   - bids: {datatype: figures, desc: about, suffix: T1w}
   - custom: boilerplate
-    path: '{reportlets_dir}/logs'
+    path: '{output_dir}/logs'
     bibfile: ['fmriprep', 'data/boilerplate.bib']
     caption: |
       <p>We kindly ask to report results preprocessed with this tool using the following boilerplate.</p>
@@ -193,6 +193,6 @@ sections:
       </p>
     title: Methods
   - custom: errors
-    path: '{reportlets_dir}/sub-{subject}/log/{run_uuid}'
+    path: '{output_dir}/sub-{subject}/log/{run_uuid}'
     captions: <em>NiReports</em> may have recorded failure conditions.
     title: Errors