Skip to content

Add splitting functionality to SpikeInterface curation #26

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.

Sign up for GitHub

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

Jump to bottom
Draft
alejoe91 wants to merge 245 commits into
base: curation-pydantic
Choose a base branch
from
Draft

Add splitting functionality to SpikeInterface curation #26

alejoe91 wants to merge 245 commits into from

Conversation

@alejoe91
Copy link
Owner

@alejoe91 alejoe91 commented Mar 26, 2025 •
edited
Loading

  • add split_units to curation model
  • add function to split a BaseSorting object
  • add split_units to SortingAnalyzer
  • add split to apply_curation
  • add _split_extension_data to extensions
  • add split_mode to curation model
  • fix handling of multi-segment

samuelgarcia and others added 30 commits October 24, 2024 12:48
@samuelgarcia
instllation using uv
f106681
@samuelgarcia @zm711 @JoeZiminski
Merci Zach and Joe
888c785 
Co-authored-by: Zach McKenzie <[email protected]>
Co-authored-by: Joe Ziminski <[email protected]>
@samuelgarcia
updates
971ce45
@samuelgarcia
work around
e298384
@samuelgarcia
job_kwargs
081a54a
@samuelgarcia @zm711
merci zach
286226c 
Co-authored-by: Zach McKenzie <[email protected]>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
ef2c00c 
for more information, see https://pre-commit.ci
@max-c-lim
Implemented RT-Sort as an external sorter
a42b572
@max-c-lim
Added reference and sorter test suite
4f0bee9
@max-c-lim
Updated installation
a38d2f8
@max-c-lim
Merge branch 'main' into rt-sort
f3c19ce
@max-c-lim
Merge branch 'main' into rt-sort
996f597
@zm711
Merge branch 'main' into rt-sort
9f2a992
@alejoe91
Merge branch 'main' into rt-sort
7fa402c
@jakeswann1
Add multi-segment support for amplitudes widget
4e65593
@alejoe91
wip
677f90c
@jakeswann1
Update base raster widget and children to handle multi-segment
2539a89
@jakeswann1
Retain sortingview compatibility
11a1845
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
16e6272 
for more information, see https://pre-commit.ci
@jakeswann1
Merge branch 'main' into main
c4fece4
@anoushkajain
Enhance CurationModel: Add split_units validation
86a3ab4
@alejoe91
Add splitting sorting to curation format
d4e0f84
@alejoe91
(wip) Add split_units to SortingAnalyzer
c7316bb
@alejoe91
Add split_units to sorting analyzer
2e21923
@alejoe91
Propagate SortingAnalyzer.split_units to apply_curation
4afdb80
@alejoe91
Extend CurationModel tests
62bfb7f
@alejoe91
Add analyzer split to curation tests
40fe01b
@alejoe91
wip: add split tests in postprocessing
ca6f2e0
@alejoe91
wip - modify model
58b62fb
@alejoe91
Fix tests and cleanup apply_curation
09a379e
h-mayorquin and others added 30 commits July 16, 2025 15:54
@h-mayorquin
add failing tests
b6e3536
@h-mayorquin
fix name in test
bdd242b
@h-mayorquin
fix
980fd85
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
153ad2e 
for more information, see https://pre-commit.ci
@alejoe91
Merge branch 'main' into installation_docs
03b4310
@alejoe91
Merge pull request SpikeInterface#4061 from alejoe91/fix-4044
dc29fc2 
Cast samples to int64 after round
@alejoe91
Merge pull request SpikeInterface#4060 from alejoe91/fix-4045
33308f8 
Check for remote paths in check_paths_relative
@alejoe91
Merge pull request SpikeInterface#4069 from h-mayorquin/fix_one_box
0711dab 
Fix one box reading in spikeglx
@alejoe91
Merge branch 'main' of github.com:SpikeInterface/spikeinterface into …
b937999 
…installation_docs
@zm711
remove tests for save/load folder
0f41fac
@alejoe91
Fix installation instructions and check your install
9b973cf
@alejoe91
Merge pull request SpikeInterface#4033 from zm711/update-deps
d34fdce 
More deprecation updates for version 0.103.0
@h-mayorquin
add comment
79968cf
@alejoe91
Update naming and delet os-specific conda env yaml
a83b166
@alejoe91
Add sampling_frequency_max_diff to aggregate_sortings
9aff21d
@alejoe91
Merge pull request SpikeInterface#4024 from DradeAW/patch-2
2ef315f 
Fix sampling rate issue when aggregating
@zm711
minor cleanup+test footnote
0490d5d
@zm711
make Windows shell explanation clearer.
2eab14b
@alejoe91
Merge pull request SpikeInterface#3503 from samuelgarcia/installation…
09a383b 
…_docs

Installation tips using uv
@alejoe91
Merge pull request SpikeInterface#3597 from max-c-lim/rt-sort
7dffef2 
Implemented RT-Sort as an external sorter
@alejoe91
Fix conflicts
3f5996b
@alejoe91
Solve conflicts, add missing docs, check consistency in analyzer.spli…
87d4e36 
…t_units
@alejoe91
Add splitting mode and warning for template split_extension_data
a2270cd
@alejoe91
Imorove logic in new unit ids for splits
084419d
@alejoe91
Merge pull request SpikeInterface#4012 from h-mayorquin/sorting_extra…
6b75bd3 
…ctors_spiking_times

Add `get_unit_spike_train_in_seconds` to sorting extractor
@alejoe91
Add property propagation to merge/split sorting
42f6522
@alejoe91
Improve readibility on generate_unit_ids_for_split
c2461fd
@alejoe91
remove comment
40c922f
@alejoe91 @samuelgarcia
Update src/spikeinterface/postprocessing/template_similarity.py
0a3be66 
Co-authored-by: Garcia Samuel <[email protected]>
@alejoe91
Merge branch 'main' into splitting-curation
c16dead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.