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

Minimum pitch and specifying channel maps (difference from v.2.0) #396

Closed
shiralupkin opened this issue May 7, 2021 · 4 comments · Fixed by #595
Closed

Minimum pitch and specifying channel maps (difference from v.2.0) #396

shiralupkin opened this issue May 7, 2021 · 4 comments · Fixed by #595

Comments

@shiralupkin
Copy link

shiralupkin commented May 7, 2021

Hi,

I am new to kilosort and have a few (related) questions about how to specify pitch (vertical and horizontal) in the mapping files and about the pitch minimums (implicit for vertical, explicit for horizontal) in v3.

  • My understanding is that vertical pitch is determined based off of the difference between the values specified in ycoords and xcoords for horizontal. My first question is to confirm that the units for the coordinates are in um, right? (as opposed to, say, mm)

  • There is a minimum horizontal pitch of 5, and therefore the program defaults to 5 if anything less is given. What do you do for single linear probes where there is no horizontal spacing (I am using 32-channel linear vprobes with 50um spacing between contacts)? Does the fact that kilosort defaults to 5 pose a problem for these types of electrodes?

  • In v3, the interpolation on line 14 of shift_batch_on_disk2 fails if the vertical pitch is anything smaller than 3 (I tested it and 3 works fine, 1 and 2 fail). I think I saw that another person noted an issue with this step a few days ago. Is there a minimum vertical pitch requirement?

  • In v2.0, neither of these pitch requirements seem to come into play (you can input a horizontal pitch of 0 and a vertical pitch of 1 and the program will run without issue). So, what role does pitch play (vertical and/or horizontal) in the respective versions? Do the minimums in v3 preclude users whose probes do not meet them?

Thank you for this great program!

@marius10p
Copy link
Contributor

The pitch is used only in v3, and only to determine how to group channels for the clustering algorithm. V2.0 uses a different clustering algorithm. As long as it runs, it should be fine.

@shiralupkin
Copy link
Author

shiralupkin commented May 11, 2021

So, if I run v3, should I just leave the default horizontal pitch and just plug in 50 for vertical? Or, should I stick to v2 for the moment?

Thanks again!

@marius10p
Copy link
Contributor

For your probe you can use 100um for the vertical and that should roughly be what Kilosort determines.

@shiralupkin
Copy link
Author

shiralupkin commented May 11, 2021 via email

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 a pull request may close this issue.

2 participants