Skip to content
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

missing API documentation on drift correction arguments #3541

Open
WeissShahaf opened this issue Nov 17, 2024 · 1 comment
Open

missing API documentation on drift correction arguments #3541

WeissShahaf opened this issue Nov 17, 2024 · 1 comment

Comments

@WeissShahaf
Copy link

in the API spikeinterface.sortingcomponents.motion.estimate_motion, for "dredge_ap":
batch_size: int
"Size of batch for the convolution. Increasing this will speed things up dramatically on GPUs and sometimes on CPU as well."

but there is no default batch_size stated

@chrishalcrow
Copy link
Collaborator

Hello,
The API docs are a little confusing here: https://spikeinterface.readthedocs.io/en/latest/api.html#spikeinterface.sortingcomponents.motion.estimate_motion
Under ‘method’, there are the possible arguments for each method, and these are all different. Only decentralized that has a batch_size arg, so changing this will have no effect on dredge_ap motion estimation.

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

No branches or pull requests

2 participants