I'm currently testing out the dev branch, which combines the changes from #445 and #407, and I noticed that the cubids group output didn't include the fMRI scans. There wasn't any kind of warning that these files weren't being grouped, because we have a try/except statement get_param_groups_dataframes that just skips over entity sets that don't successfully group, without any kind of warning.
|
for entity_set in entity_sets: |
|
try: |
|
( |
|
labeled_file_params, |
|
param_summary, |
|
modality, |
|
) = self.get_param_groups_from_entity_set(entity_set) |
|
except Exception: |
|
continue |
I'm currently testing out the
devbranch, which combines the changes from #445 and #407, and I noticed that thecubids groupoutput didn't include the fMRI scans. There wasn't any kind of warning that these files weren't being grouped, because we have a try/except statementget_param_groups_dataframesthat just skips over entity sets that don't successfully group, without any kind of warning.CuBIDS/cubids/cubids.py
Lines 1211 to 1219 in b172dd3