Skip to content

Add install check for components sorters#4574

Open
chrishalcrow wants to merge 7 commits intoSpikeInterface:mainfrom
chrishalcrow:add-components-install-errors
Open

Add install check for components sorters#4574
chrishalcrow wants to merge 7 commits intoSpikeInterface:mainfrom
chrishalcrow:add-components-install-errors

Conversation

@chrishalcrow
Copy link
Copy Markdown
Member

@chrishalcrow chrishalcrow commented May 4, 2026

Adds groups for the internal sorters.
Adds a package installation check when you run LUPIN, spykingcircus2 and tridesclous2, explaining how to install using the new groups.

Example error:

Exception: The sorter tridesclous2 is not installed. Please install it with:
        pip install 'spikeinterface[tridesclous2]'
Or, if you have cloned SpikeInterface locally, using:
        pip install '.[tridesclous2]'

@chrishalcrow chrishalcrow requested a review from samuelgarcia May 4, 2026 09:38
@chrishalcrow chrishalcrow added sortingcomponents Related to sortingcomponents module packaging Related to packaging/style labels May 4, 2026
@chrishalcrow chrishalcrow requested a review from yger May 4, 2026 09:39
Comment thread pyproject.toml Outdated
@alejoe91
Copy link
Copy Markdown
Member

alejoe91 commented May 4, 2026

Seems like lupin tests in internal sorters are failing due to installation

@chrishalcrow
Copy link
Copy Markdown
Member Author

I missed the sorters module in #4573 - d'oh.

Copy link
Copy Markdown
Collaborator

@h-mayorquin h-mayorquin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging Related to packaging/style sortingcomponents Related to sortingcomponents module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants