-
Notifications
You must be signed in to change notification settings - Fork 237
Random spike selection new methods #4276
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
base: main
Are you sure you want to change the base?
Random spike selection new methods #4276
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 |
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.