-
Notifications
You must be signed in to change notification settings - Fork 61
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
Rename spike_detector to something more descriptive #1642
Comments
I agree that the name leaks the implementation details. But, a term should be
I have no good solution to offer, sadly, so suggest away ;) |
just |
That's literally the name of the C++ class |
Then let's rename the Python wrapper and make |
There you go. |
The term
spike_detector
is confusing. Proposed renamings:membrane_threshold_detector
mtd
The text was updated successfully, but these errors were encountered: