-
Notifications
You must be signed in to change notification settings - Fork 186
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
sorting with kilosrt3 not working #2416
Comments
Hi @neuronzoo, it would be super helpful to see the full script you are trying to run that triggered this error. The |
For sure - and thanks again for the help -> Here is the script: import spikeinterface as si ss.Kilosort3Sorter.set_kilosort3_path("C:...\SpikeInterface\Kilosort-main\Kilosort-main") recording_file_oe1 = r"D:...\TAS\2023-09-24" recording_concat = si.concatenate_recordings([recording1, recording2]) we = sc.WaveformExtractor.create(recording_concat, folder = r"D:\...\Stitching_trial_WT_R06_092423_POS_TAS_prA\new") sorting_concat = ss.run_sorter("kilosort3", recording_concat, output_folder="C:\output_folder") |
One more question. I'm looking back at your log you shared and the lines aren't matching up with the current source code. What version of spikeinterface are you using? The changes to help Windows recently occurred so I want to make sure you have a recent enough version. Basically could I get an si version and python version too. Thanks! |
Sorry for the delay! |
@neuronzoo |
Happy to! Any chance you can tell me which line to run in the env? Also, I hope everything should be backwards compatible :X I rely heavily on SI for analyses now :) |
From 97-99 it should be basically all backward compatible. I can't think of any breaking changing. A few functions have been changed to be faster/better, but the structure itself is all still there. So you may need to change a few lines of your code to do something (for example we've gotten rid of You would either do: pip install -U spikeinterface which will install the newest pypi version or, cd github/spikinterface
pip install -e .[full,widgets] to do from source (of course you'd have to download the source code first if you want to do this). |
Cool. Will try that and let you know! Thanks again :) |
Any updates? Just want to see if the update worked. There's actually an even newer version as of today, so if you haven't updated it's an even better time to. :) |
I'll close this for inactivity, but feel free to reopen if the issue comes up. Note that there are spike hole changes to KS3 that may be of interest. |
I am using SI a lot for analyzing NPX data. I am now trying to use SI to sort two concatenated NPX sessions acquired using Open Ephys. So far I've used KS3 directly through matlab with no issues.
I use
sorting_concat = ss.run_sorter("kilosort3", recording_concat, output_folder="C:\output_folder")
The code seems to run, but does not produce any results, running for more than 12 hours before being terminated by me (each session woud normally take 2 hrs to run from Matlab). When it runs, GPU usage is at baseline, so I don't think it does any sorting. Trying to debug for the culprit line, I find that running line 196 in basesorter.py takes eternity and does not complete:
cls._setup_recording(recording, sorter_output_folder, sorter_params, verbose)
I also get the attached printout. I tried using a non-stitched session, with no luck. Thank you!
printout_orting_stitched_sessions.txt
The text was updated successfully, but these errors were encountered: