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

Little fix for 'eig did not converge' crash (Kilosort 2.0) #560

Closed
wants to merge 1 commit into from

Conversation

DradeAW
Copy link

@DradeAW DradeAW commented Aug 2, 2023

These changes do not change the default Kilosort 2.0 behaviour, but adds alternatives in case of a crash to try to make the run successful.

If these changes still provide a crash, in my experience increasing the 'NT' parameter helps :)

@abisi
Copy link

abisi commented Aug 3, 2023

Hello, I've had this issue ("EIG did not converge at index ") just recently in runTemplates. Did you get why this occurs?
Also, from what value to what value did you increase the NT parameter?

Thanks!
Axel

@DradeAW
Copy link
Author

DradeAW commented Aug 3, 2023

Hi!

My understanding is that computing the eigenvectors and eigenvalues for such large arrays is complicated.
Sometimes the method employed by Kilosort fails (especially for long and heavy-channel recordings)

So trying an alternative way can be useful!

Here is the default value for NT:

ops.NT = 64*1024+ ops.ntbuff; % must be multiple of 32 + ntbuff. This is the batch size (try decreasing if out of memory).

I changed it to 3x this value, and for my recording it worked. You can also try 10x the value, it shouldn't have too much of an effect on the sorting efficacy.

@abisi
Copy link

abisi commented Aug 4, 2023

Hello,

Thank you for your answer! I'll try both your proposed changes and changing ops.NT to see what would fix this issue in my case (2h long recordings with all bank channels).

So far, going from Matlab 2023a back to Matlab 2021b - which I was using - fixed the issue.

Best,
Axel

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

Successfully merging this pull request may close these issues.

None yet

2 participants