SpikeInterface / SpikeInterface/spikeinterface
Add `pandas` to main dependencies?
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 847
- Forks
- 280
- Avg merge
- 3d 9h
- Merged PRs (30d)
- 29
Description
(AFAICT) pandas is required for sorting in kilosort as this line here as BasePhyKilosortSortingExtractor requires it.
As this should be quite a common use case (sorting with kilosort4 locally), should we add pandas to the core dependencies? I don't think it's particularly heavy (?)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the pandas use in src/spikeinterface/sorters/external/kilosortbase.py and the requirement in src/spikeinterface/extractors/phykilosortextractors.py. Then inspect the project’s core dependency declaration and confirm whether local Kilosort4 use reaches these entry points without pandas installed. Done means the dependency policy is resolved and the relevant dependency metadata reflects that decision.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- pandas, python
- Domain
- build-system
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100