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

kilosort import with few cells #156

Open
agbondy opened this issue Nov 9, 2020 · 1 comment
Open

kilosort import with few cells #156

agbondy opened this issue Nov 9, 2020 · 1 comment

Comments

@agbondy
Copy link
Contributor

agbondy commented Nov 9, 2020

We are tracking yields over time in chronic recordings, and this means we intentionally record some sessions when they are very few cells. Kilosort import only imports the "good" channels which in these cases can be very few, even less than 10 channels. This makes a mess of the kilosort import which expects there to be at least nSitesDir.*2 + 1 channels (this is typically 14 in our case). The crash happens at the line shown in the picture below. In words, the crash is happening because the ap_raw.jrc file only has three channels but the detection algorithm is trying to load nSites.*2 + 1 of them. Obviously this is a very nontypical case, but it would be good to have sensible behavior here, nonetheless -- at the very least a human-readable error.

Image from iOS

@aliddell
Copy link
Collaborator

If you want to try this with the latest string of commits and show me where it's crashing (or send me the data and let me find it myself), I'll get on it.

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

2 participants