Skip to content

Implement curation_callback and set_external_curation#241

Merged
samuelgarcia merged 8 commits intoSpikeInterface:mainfrom
alejoe91:curation-callback
Mar 13, 2026
Merged

Implement curation_callback and set_external_curation#241
samuelgarcia merged 8 commits intoSpikeInterface:mainfrom
alejoe91:curation-callback

Conversation

@alejoe91
Copy link
Member

@alejoe91 alejoe91 commented Feb 20, 2026

Fixes #239

Option to pass a custom curation callback with kwargs. This can entirely replaced the Submit to parent functionality in panel, since it can be taken care of by the parent app directly.

See new test test_panel_embedded.py for demonstration!

Similarly, this PR enables bi-directional communication by exposing a set_external_curation to the panel backend. This will apply the external curation if the curationview is available, notify a manual curation changed, and refresh labels in the units table.

Replaces #228, since all communication can be now handled externally.

tabs.stylesheets = []


def set_external_curation(self, curation_data):
Copy link
Member Author

Choose a reason for hiding this comment

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

@chrishalcrow could this be needed by the Qt backend too? e.g., UnitRefine or similar could use it?

@alejoe91 alejoe91 changed the title Curation callback concept Implement curation_callback and set_external_curation Feb 25, 2026
self.notify_manual_curation_updated()
self.notifier.notify_active_view_updated()

def _panel_update_labels(self):
Copy link
Member Author

@alejoe91 alejoe91 Feb 27, 2026

Choose a reason for hiding this comment

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

Make full refresh instead, which includes the labels

Copy link
Member

Choose a reason for hiding this comment

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

Do you plan to do this or can I merge this PR ?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we can merge

@alejoe91 alejoe91 added this to the 0.13.0 milestone Mar 6, 2026
@samuelgarcia samuelgarcia merged commit 50e02ee into SpikeInterface:main Mar 13, 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.

Add option to pass curation_callback function

2 participants