wave_clus icon indicating copy to clipboard operation
wave_clus copied to clipboard

Minor bug with batch input to Do_clustering.m

Open ckfaber opened this issue 2 years ago • 1 comments

I'm inputting a cell array of _spikes.mat files to Do_clustering.m. While the function is running through, the function does not give an accurate message on progression through the 'times' files - relevant code is in lines 166-180 of Do_clustering, specifically:

    `    disp(sprintf('%d of %d ''times'' files finished.',count_new_times(fnum, filenames),Nfiles))`

Mine just keeps saying 0 of 16 'times' files finished on every iteration of the loop. Should be an easy fix, hopefully!

Thanks for such great functions and documentation!

ckfaber avatar Sep 12 '22 19:09 ckfaber

Hi, that is a timing issue related to how the parallel processing is done and the times associated to the files. How is done, makes sense for slower computers. I'm not actively fixing that type of non essential bugs, but I can accept a workaround if you make a pull request.

ferchaure avatar Sep 13 '22 23:09 ferchaure