csn-le / wave_clus

A fast and unsupervised algorithm for spike detection and sorting using wavelets and super-paramagnetic clustering
127 stars 66 forks source link

Batch processing, file extension missing #120

Closed YveKr closed 5 years ago

YveKr commented 5 years ago

I just have a minor point about the batch processing routine, it does not work until you add '.mat' extensions to the 'spikes' files and to the 'times' files saved during batch processing. One needs to add these during the saving stage in both batch-files to get correct output and figures. Thanks for the new version.

ferchaure commented 5 years ago

that is odd, Matlab adds its extension automatically to the files, which Matlab version are you using?

YveKr commented 5 years ago

R2016b

YveKr commented 5 years ago

it resulted in something similar to this post, subplots not showing up for the figures because not able to open 'times_' : https://github.com/csn-le/wave_clus/issues/51#issuecomment-333220060

only a minor issue though

ferchaure commented 5 years ago

Does the raw data files a dot in the name?

YveKr commented 5 years ago

Yes, indeed this might be the issue. But after adding '.mat' to force it, everything works fine.

ferchaure commented 5 years ago

I would recommend remove the dot in the filenames, it can be annoying for a lot of applications.

YveKr commented 5 years ago

I know... it's just that I got many short files (a few 100s) from different people and they all have info such as recording coordinates, durations, etc. in the file names. it's certainly not ideal, but once sorted, I won't use the RAW data files anymore. Thanks