Skip to content
This repository has been archived by the owner on May 4, 2023. It is now read-only.

Minor bug with batch input to Do_clustering.m #219

Open
ckfaber opened this issue Sep 12, 2022 · 1 comment
Open

Minor bug with batch input to Do_clustering.m #219

ckfaber opened this issue Sep 12, 2022 · 1 comment

Comments

@ckfaber
Copy link

ckfaber commented Sep 12, 2022

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!

@ferchaure
Copy link
Member

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants