Skip to content

Commit 717f500

Browse files
authored
Merge pull request #12 from neurorepro/adds_populate_intended_for
BF: In `find_compatible_fmaps_for_run`, actually check if the first element of the parameters list is a string. Thanks, @neurorepro
2 parents 0f38e5e + 88e93eb commit 717f500

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@
55
.idea/
66
venvs/
77
_build/
8+
build/

heudiconv/bids.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -687,7 +687,7 @@ def find_compatible_fmaps_for_run(json_file, fmap_groups, matching_parameters):
687687
for param in matching_parameters:
688688
fm_info = get_key_info_for_fmap_assignment(fm_group[0], param)
689689
# for the case in which key_info is a list of strings:
690-
if type(json_info[param][0]):
690+
if isinstance(json_info[param][0], str):
691691
compatible = json_info[param] == fm_info
692692
else:
693693
# allow for tiny differences between the affines etc

0 commit comments

Comments
 (0)