Random spike selection new methods#4276
Random spike selection new methods#4276tayheau wants to merge 20 commits intoSpikeInterface:mainfrom
Conversation
for more information, see https://pre-commit.ci
|
Hi @tayheau. |
|
Hey @samuelgarcia, yeah of course, just to summarize here quickly: So the method is temporal bin sampling and i expressed it as a sub sampling problem.
I m not sure if my explainations are very clear, that's why i prefer to write it here in a first time to have some feed back on it. |
|
By the way @samuelgarcia, @alejoe91 , perhaps we can add the |
Yes it definitely deserve some more explanation! :) |
|
I don't think we have a specific place yet, but maybe it's worth adding a https://spikeinterface.readthedocs.io/en/latest/modules/core.html#sortinganalyzer |
| @@ -410,11 +410,17 @@ def get_segment_durations(sorting: BaseSorting, segment_indices: list[int]) -> l | |||
| sorting : BaseSorting | |||
| The sorting object containing spike data | |||
|
|
|||
There was a problem hiding this comment.
I am a master reviewer.
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
So following #4251, a PR with a few new sampling methods for
random_spikes_selection:percentagecapmaximum_ratecaptemporal_binssampling for a better temporal representation of the original population, i tried to fully vectorize it so the temporal bin sampling problem is dual with a subsorting problem.I also added to possibility to
get_segment_durationsto setsegment_indicestoNone.