Skip to content

Add splitting functionality to curation and SortingAnalyzer #3817

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

alejoe91
Copy link
Member

Depends on #3760

Implements splitting functionality in:

@alejoe91 alejoe91 added the curation Related to curation module label Mar 28, 2025
Comment on lines +597 to +599
old_template = arr[self.sorting_analyzer.sorting.ids_to_indices([split_unit_id])[0], ...]
new_indices = np.array([new_unit_ids.index(unit_id) for unit_id in new_splits])
new_array[new_indices, ...] = np.tile(old_template, (len(new_splits), 1, 1))
Copy link
Member Author

Choose a reason for hiding this comment

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

@samuelgarcia this needs to be discussed. What should we do if the waveforms extension is not there? The current behavior is copying, but we mught want to force recompute here

@samuelgarcia
Copy link
Member

This is amazing camarade.

I would do first a PR to focus only on the format. And then merge this one.
What do you think ?

@alejoe91
Copy link
Member Author

I did camarade! #3760

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
curation Related to curation module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants