Skip to content

Commit 42472d9

Browse files
committed
FIX: Update reference-mask reportlet queries
1 parent c916abe commit 42472d9

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed

petprep/data/reports-spec-pet.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ sections:
99
- bids: {datatype: figures, desc: carpetplot, suffix: pet}
1010
- bids: {datatype: figures, desc: confoundcorr, suffix: pet}
1111
- bids: {datatype: figures, desc: coreg, suffix: pet}
12-
- bids: {datatype: figures, desc: refmask, suffix: pet}
12+
- bids: {datatype: figures, desc: ref, suffix: pet}
1313
caption: Reference region mask overlaid on the PET reference and anatomical data.
1414
static: false
1515
subtitle: Reference mask check

petprep/data/reports-spec.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ sections:
138138
caption: PET to anatomical alignment check
139139
static: false
140140
subtitle: Additional PET Visualizations
141-
- bids: {datatype: figures, desc: refmask, suffix: pet}
141+
- bids: {datatype: figures, desc: ref, suffix: pet}
142142
caption: Reference region mask overlaid on the PET reference and anatomical.
143143
static: false
144144
subtitle: Reference mask check

petprep/workflows/pet/tests/test_fit.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -223,8 +223,8 @@ def test_refmask_report_connections(bids_root: Path, tmp_path: Path, pvc_method)
223223

224224
assert 'ds_refmask_wf.ds_refmask' in wf.list_node_names()
225225
ref_ds = wf.get_node('ds_refmask_wf').get_node('ds_refmask')
226-
assert ref_ds.inputs.desc == 'refmask'
227-
assert ref_ds.inputs.ref == 'cerebellum'
226+
assert ref_ds.inputs.desc == 'ref'
227+
assert ref_ds.inputs.label == 'cerebellum'
228228
assert 'func_fit_reports_wf.pet_t1_refmask_report' in wf.list_node_names()
229229
reports_node = wf.get_node('func_fit_reports_wf')
230230
edge = wf._graph.get_edge_data(wf.get_node('outputnode'), reports_node)
@@ -360,8 +360,8 @@ def test_init_refmask_report_wf(tmp_path: Path):
360360
wf = init_refmask_report_wf(output_dir=str(tmp_path), ref_name='test')
361361
assert 'mask_report' in wf.list_node_names()
362362
ds = wf.get_node('ds_report_refmask')
363-
assert ds.inputs.desc == 'refmask'
364-
assert ds.inputs.ref == 'test'
363+
assert ds.inputs.desc == 'ref'
364+
assert ds.inputs.label == 'test'
365365
assert ds.inputs.suffix == 'pet'
366366

367367

@@ -370,7 +370,11 @@ def test_reports_spec_contains_refmask():
370370
for fname in ('reports-spec.yml', 'reports-spec-pet.yml'):
371371
spec = yaml.safe_load((data.load.readable(fname)).read_text())
372372
pet_section = next(s for s in spec['sections'] if s['name'] == 'PET')
373-
assert any(r.get('bids', {}).get('desc') == 'refmask' for r in pet_section['reportlets'])
373+
assert any(
374+
r.get('bids', {}).get('desc') == 'ref'
375+
and 'label' not in r.get('bids', {})
376+
for r in pet_section['reportlets']
377+
)
374378

375379

376380
def test_refmask_reports_omitted(bids_root: Path, tmp_path: Path):

0 commit comments

Comments
 (0)