Skip to content

Main channel template#233

Merged
chrishalcrow merged 5 commits intoSpikeInterface:mainfrom
samuelgarcia:main_channel_template
Feb 23, 2026
Merged

Main channel template#233
chrishalcrow merged 5 commits intoSpikeInterface:mainfrom
samuelgarcia:main_channel_template

Conversation

@samuelgarcia
Copy link
Copy Markdown
Member

@samuelgarcia samuelgarcia commented Feb 10, 2026

Plot the upsampled template on main channel used to compute template metrics.

Not a game changer viewbut can be usefull.

Notes: the challenge was more on spikeinterface side to save and retrieve theses infos in the analyzer, (after internal and non ending technical debates)

image

@chrishalcrow @alejoe91 : the name "mainchannel" is bad. give me other ideas.

@alejoe91
Copy link
Copy Markdown
Member

template_shape?

@chrishalcrow
Copy link
Copy Markdown
Member

Amazing, Sam!

When you have smooted and upsampled, it gets a bit messy: I think the white of the upsampled gets mixed in with the widths and peaks. Maybe it needs to be in another colour (or style?). And we either need a legend, or an explanation in the [?].

Screenshot from 2026-02-11 10-41-25

@chrishalcrow
Copy link
Copy Markdown
Member

maintemplate
extremaltemplate
?

@alejoe91
Copy link
Copy Markdown
Member

@chrishalcrow renamed maintemplate, made it back-compatible (if you don;t have peaks info, it will onlty show the main channel) and made original template dashed, width/height lines dotted, and troughs as downward triangles, peaks upward triangles. Also added in docs!

image

Comment thread spikeinterface_gui/maintemplateview.py Outdated
Comment thread spikeinterface_gui/controller.py Outdated
Comment thread spikeinterface_gui/controller.py Outdated
Comment thread spikeinterface_gui/maintemplateview.py
'spikerate': 'spikerateview.py',
'metrics': 'metricsview.py',
'mainsettings': 'mainsettingsview.py',
p: f"{p}view.py" for p in all_possible_class_views.keys()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Magnifico

@alejoe91
Copy link
Copy Markdown
Member

@chrishalcrow template_metrics dependency is now optional as suggested :)

@alejoe91
Copy link
Copy Markdown
Member

Let's merge?

@chrishalcrow chrishalcrow merged commit 9fd9738 into SpikeInterface:main Feb 23, 2026
1 check passed
@chrishalcrow
Copy link
Copy Markdown
Member

merci

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.

3 participants