-
Notifications
You must be signed in to change notification settings - Fork 242
Open
Description
In earlier versions of SpikeInterface, the SortingAnalyzer's recording's and sorting's number of segments did not need to match.
For example, we were recording for several trials and having several recording files. We give the merged (concatenated cat) file to our favorite spike sorter. We get a sorting result out in global times.
Creating a SortingAnalyzer
calls
| if has_exceeding_spikes(sorting=sorting, recording=recording): |
which does
spikeinterface/src/spikeinterface/curation/remove_excess_spikes.py
Lines 30 to 32 in 25940b4
| assert ( | |
| sorting.get_num_segments() == recording.get_num_segments() | |
| ), "The sorting and recording objects must have the same number of samples!" |
and complains for out multi-segment recordings with single-segment sorting.
I think we should treat the recording as a single recording.
The error message should mean "segments" instead of "samples".
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels