spikeinterface icon indicating copy to clipboard operation
spikeinterface copied to clipboard
verified publisher icon SpikeInterface

Add multi-segment capability to BaseRasterWidget and children

Open jakeswann1 opened this issue 9 months ago • 2 comments

Adds the option to pass a list of segment indices to the AmplitudesWidget, DriftRasterMapWidget, and RasterWidget to plot across multiple segments, by updating how the base widget handles plotting data. Maintains current default behaviour and SortingView capability. resolves #3801

jakeswann1 avatar Mar 25 '25 17:03 jakeswann1

Multi-segment plots would look like this:

image image image

jakeswann1 avatar Mar 25 '25 17:03 jakeswann1

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 could be improved here!

jakeswann1 avatar Apr 30 '25 11:04 jakeswann1

@jakeswann1 can you apply the changes from Chris? We cannot push to your fork since you made the PR from main

alejoe91 avatar Jul 03 '25 14:07 alejoe91

Thanks @jakeswann1

LGTM! @chrishalcrow wanna take a final look?

alejoe91 avatar Jul 04 '25 13:07 alejoe91

One tiny change, then will approve :)

chrishalcrow avatar Jul 04 '25 14:07 chrishalcrow

Moved to #4035 so it can be merged properly

jakeswann1 avatar Jul 06 '25 22:07 jakeswann1