wiheto / teneto

Temporal Network Tools
GNU General Public License v3.0
85 stars 26 forks source link

error in params to make_parcellation #35

Closed granitz closed 5 years ago

granitz commented 5 years ago

When I run make_parcellation I get the following error which seems to be removed when I comment 'sidecar'.

~/anaconda3/lib/python3.6/site-packages/teneto/classes/bids.py in make_parcellation(self, parcellation, parc_type, parc_params, network, update_pipeline, removeconfounds, tag, njobs, clean_params) 859 860 for i,f in enumerate(files): --> 861 self._run_make_parcellation(f,i,tag,parcellation,parc_name,parc_type,parc_params) 862 863 if update_pipeline == True:

~/anaconda3/lib/python3.6/site-packages/teneto/classes/bids.py in _run_make_parcellation(self, f, i, tag, parcellation, parc_name, parc_type, parc_params) 878 fsave, tag, 'parcellation', 'roi') 879 roi = teneto.utils.make_parcellation( --> 880 f, parcellation, parc_type, parc_params) 881 # Make data node,time 882 roi = roi.transpose()

~/anaconda3/lib/python3.6/site-packages/teneto/utils/utils.py in make_parcellation(data_path, parcellation, parc_type, parc_params) 863 if parc_type == 'sphere': 864 parcellation = load_parcellation_coords(parcellation) --> 865 seed = NiftiSpheresMasker(np.array(parcellation), **parc_params) 866 data = seed.fit_transform(data_path) 867 elif parc_type == 'region':

TypeError: init() got an unexpected keyword argument 'description'

wiheto commented 5 years ago

What are you passing in parc_params when this error occurs?

granitz commented 5 years ago

These are the params: nilearn_params = {'standardize': True,'low_pass': 0.1,'high_pass': 0.01,'t_r':2.396}

wiheto commented 5 years ago

Could you please explain what “removing sidecar” means. Because the error you paste has nothing about sidecar in it.

18 mars 2019 kl. 01:39 skrev granitz notifications@github.com:

These are the params: nilearn_params = {'standardize': True,'low_pass': 0.1,'high_pass': 0.01,'t_r':2.396}

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

wiheto commented 5 years ago

This seems to be solved now, right?

granitz commented 5 years ago

Yes, this is resolved now, thank you!

wiheto commented 5 years ago

Great!