Skip to content

Commit 6dbfbcb

Browse files
oestebanmgxd
andauthored
Apply suggestions from code review
Co-authored-by: Mathias Goncalves <[email protected]>
1 parent 82fd9b2 commit 6dbfbcb

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

sdcflows/utils/wrangler.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ def find_estimators(*, layout, subject, fmapless=True, force_fmapless=False):
260260

261261
b0_ids = tuple()
262262
with suppress(BIDSEntityError):
263-
b0_ids = layout.get_B0FieldIdentifiers()
263+
b0_ids = layout.get_B0FieldIdentifiers(**base_entities)
264264

265265
for b0_id in b0_ids:
266266
# Found B0FieldIdentifier metadata entries
@@ -283,9 +283,9 @@ def find_estimators(*, layout, subject, fmapless=True, force_fmapless=False):
283283
estimators.append(e)
284284

285285
# A bunch of heuristics to select EPI fieldmaps
286-
sessions = layout.get_sessions()
287-
acqs = tuple(layout.get_acquisitions(suffix="epi") + [None])
288-
contrasts = tuple(layout.get_ceagents(suffix="epi") + [None])
286+
sessions = layout.get_sessions(subject=subject)
287+
acqs = tuple(layout.get_acquisitions(subject=subject, suffix="epi") + [None])
288+
contrasts = tuple(layout.get_ceagents(subject=subject, suffix="epi") + [None])
289289

290290
for ses, acq, ce in product(sessions or (None,), acqs, contrasts):
291291
entities = base_entities.copy()
@@ -339,7 +339,7 @@ def find_estimators(*, layout, subject, fmapless=True, force_fmapless=False):
339339
from .epimanip import get_trt
340340

341341
# Sessions may not be defined at this point if some id was found.
342-
sessions = layout.get_sessions()
342+
sessions = layout.get_sessions(subject=subject)
343343
for ses, suffix in sorted(product(sessions or (None,), fmapless)):
344344
candidates = layout.get(suffix=suffix, session=ses, **base_entities)
345345

0 commit comments

Comments
 (0)