Skip to content

Remove default num_chan (sparsity) column and add 'clear' label shortcut#244

Merged
chrishalcrow merged 3 commits intoSpikeInterface:mainfrom
alejoe91:remove-sparsity-column
Mar 9, 2026
Merged

Remove default num_chan (sparsity) column and add 'clear' label shortcut#244
chrishalcrow merged 3 commits intoSpikeInterface:mainfrom
alejoe91:remove-sparsity-column

Conversation

@alejoe91
Copy link
Copy Markdown
Member

The sparsity column is redundant and just takes up space.

Also adding the c shortcut to clear labels

@chrishalcrow
Copy link
Copy Markdown
Member

I'm in favor of this, but maybe there's a reason to include the sparsity: @samuelgarcia ??
If there is, maybe we can add it to the columns but turn it off by default?

@alejoe91
Copy link
Copy Markdown
Member Author

I'm in favor of this, but maybe there's a reason to include the sparsity: @samuelgarcia ?? If there is, maybe we can add it to the columns but turn it off by default?

I'm in favor of this, but maybe there's a reason to include the sparsity: @samuelgarcia ?? If there is, maybe we can add it to the columns but turn it off by default?

Done in last commit: just adding it to the controller.units_table does the trick!

@alejoe91 alejoe91 changed the title Remove sparsity column and add 'clear' label shortcut Remove default num_chan (sparsity) column and add 'clear' label shortcut Feb 25, 2026
@alejoe91 alejoe91 added this to the 0.13.0 milestone Mar 6, 2026
@alejoe91
Copy link
Copy Markdown
Member Author

alejoe91 commented Mar 6, 2026

@samuelgarcia approved this and he also said we can get rid of num_chan completely (which is honestly a bit useless :P)

@chrishalcrow chrishalcrow merged commit 1263de6 into SpikeInterface:main Mar 9, 2026
1 check passed
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 this pull request may close these issues.

2 participants