Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merging Clusters causes loss of valid ISIs data #34

Open
juliencarponcy opened this issue Nov 15, 2019 · 4 comments
Open

Merging Clusters causes loss of valid ISIs data #34

juliencarponcy opened this issue Nov 15, 2019 · 4 comments

Comments

@juliencarponcy
Copy link

Hello,

I have observed that ALL the ISIs < spkRefrac_merge_ms are automatically deleted when you are merging two clusters. Thus this is likely to cause the (permanent) loss of valid ISIs data <2ms (default parameter), which then, cause a very dodgy side effect:
You are very unlikely to have violation of your refractory period if you are deleting all ISIs < 2ms, and thus, a lot of users could consider clusters as "single units" just because they have merged two clusters together.
Could you please investigate this further ? Just to confirm or try to find out what I got wrong ?
(I have to perform all of my sorting again because of that. I'll let you know if I notice other side effects)

Thank you very much for your beautiful and convenient work anyway.

Best,

Julien

@juliencarponcy juliencarponcy changed the title Merging Clusters causes lost of valid ISIs data Merging Clusters causes loss of valid ISIs data Nov 15, 2019
@jamesjun
Copy link
Collaborator

jamesjun commented Nov 15, 2019 via email

@juliencarponcy
Copy link
Author

Thanks for the quick reply,

Anyway, I think that whatever the parameter you can end up with some minor issues... I'll follow up later on that and provide you a few examples. Two different problematic cases for me is, what happen when a small waveform is detected over a bigger spike, and how to deal with the merging of clusters which have been detected on two different sites. (Very dependant on the lag you can have between the two peaks). Thus indeed you don't want spkRefrac_merge_ms to be too small neither..
As I said I'll provide a few examples later.

Thanks very much,

Julien

@kouichi-c-nakamura
Copy link

Thanks James.

what happen when a small waveform is detected over a bigger spike, and how to deal with the merging of clusters which have been detected on two different sites.

Julien and I are literally talking about these issues! I could not see an easy solution.

@juliencarponcy
Copy link
Author

juliencarponcy commented Nov 18, 2019

Hi James,

I guess there is two sort of problems that we do not really have clue on how to resolve them.

The first one is when two different spikes are detected close to one another (Trace view and most left clusters on Cluster view). The problem arise in the fact that when you are merging clusters then, I feel that it gave rise to genuine merging + spike detected twice, and I don't know very much how to control for that.

The other occurs when the same spike is detected on (very) distant contacts (Black and Red clusters on cluster view). In that case also, I have observed genuine merging + double detection. For that I guess that the best chance is to play on maxDist_site_um AND/OR maxDist_site_spk_um. Is that possible to specify the first one larger that the second one, and how much you think increasing too much this parameter could impede detection of other genuine spikes occuring very close in time and space ?

In other words, it is very tricky for us to be sure of our combination between spkRefrac_ms , spkRefrac_merge_ms,maxDist_site_um , and maxDist_site_spk_um. I would be much more keen to redo my sorting if I could be more certain of the outcome of the combination of these parameters, but that's obviously a lot of combination to test and results for us would just be empirical.

Any thoughts on that ?

Thank you very much for your help,

Julien

PS : I would be happy to share a dataset if you think that could help you in having a better idea

ClusterView-SpikeDetectedOnDifferentContact
Overlapping-traces

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

No branches or pull requests

3 participants