spikes icon indicating copy to clipboard operation
spikes copied to clipboard

Added support for cluster_KSLabel.tsv'

Open peterzh opened this issue 4 years ago • 3 comments

Here I added support for loading cluster_KSLabel.tsv

peterzh avatar Apr 01 '20 13:04 peterzh

Any chance you are willing to go a step further while you're at it and add support for loading any tsv file that's found in the directory? It would need to do a bit of sanitizing, like checking that it loads, and checking that every entry in the first column is a valid cluster ID (that would be a great check to add for cluster_group and cluster_KSLabel, too). I think ideally all tsv's besides cluster_group should go into a table together, with each becoming a column, rows for cluster IDs. Basically a table that looks like the clusterview in phy. What do you think?

nsteinme avatar Apr 01 '20 14:04 nsteinme

Sure I can code that up. I'm confused as to why 'cluster_group' should not be in a table with the other cluster_* files though

peterzh avatar Apr 15 '20 13:04 peterzh

I've started working on this code, and I have another question about the current loadKSdir code:

In line 27 the code loads spike_clusters.npy into 'clu'. In line 72 this variable is then overwritten with spikeTemplates, if no cluster groups file is found. I was wondering why this happens? Why not keep clu as it is, and not overwrite with spikeTemplates?

Thanks!

peterzh avatar Apr 15 '20 14:04 peterzh