-
Notifications
You must be signed in to change notification settings - Fork 240
Add working with tetrodes doc #4055
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
Conversation
zm711
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment/typo.
zm711
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One last comment. While you address that I'll look at the actual rendering.
zm711
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
functions are rendering with italics. We need to add the :code: tag or maybe `` but I only use :code: so that's why I suggested it.
Co-authored-by: Zach McKenzie <92116279+zm711@users.noreply.github.com>
Co-authored-by: Zach McKenzie <92116279+zm711@users.noreply.github.com>
Co-authored-by: Zach McKenzie <92116279+zm711@users.noreply.github.com>
Co-authored-by: Zach McKenzie <92116279+zm711@users.noreply.github.com>
zm711
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good by me.
alejoe91
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @chrishalcrow
See my comment about CMR, the rest looks great :)
Co-authored-by: Zach McKenzie <92116279+zm711@users.noreply.github.com>
| # This is still a danger if you have two tetrodes in a bundle, which | ||
| # might pick up the same spike, but becomes less dangerous | ||
| # as the number of tetrodes in your bundle increases. | ||
| # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Genial!
zm711
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still good by me too!
Add PR giving advice about working with tetrodes. Contains
Docs: https://spikeinterface--4055.org.readthedocs.build/en/4055/tutorials/forhowto/plot_1_working_with_tetrodes.html
Also the first example of a How To guide that we create as a sphnx-gallery item (but include in the How Tos rather than the Tutorials). Is pretty easy to do!
Closes #1877