Jake Swann

Results 13 comments of Jake Swann

Hi both! I had the same TypeError today using Kilosort 4.0.7 and Spikeinterface 101.0 (from source). The only Spikeinterface functions I used prior to running the sorter were `se.read_openephys()` and...

Hi Sam, thanks! That makes a lot of sense, I'll work on a refactor

Hi @samuelgarcia, I think this is ready for review - split into a new function as suggested

Sounds good! The segment index param currently only applies to these subclasses, and isn't inherently handled by `BaseRasterWidget` - would your suggestion be to edit the base class to handle...

#3805 PR drafted - let me know what you think!

Multi-segment plots would look like this: ![image](https://github.com/user-attachments/assets/e8e1d861-cdfd-4fdb-be52-9565595e4cb7) ![image](https://github.com/user-attachments/assets/cf44cb9f-3db4-4988-bff6-95e241140946) ![image](https://github.com/user-attachments/assets/a11e5e9e-0924-4059-b4b4-7cc6545b9626)

Hey @chrishalcrow - sorry for the delay working on this, I've been away for a while. Think I've addressed your comments, let me know if there's anything else you think...

Moved to #4035 so it can be merged properly

Thanks for the reply! I can install from source fine, but the pypi install is still failing on my machine, if that helps at all

Scikit-learn is not a dependency either - how do we want to handle this? It would be needed for both the prediction and training steps, if we implement that too