-
Notifications
You must be signed in to change notification settings - Fork 190
[BEP32 fix] Reorder required metadata files for microephys sessions and clarify channel information #2153
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
…ify channel information
|
|
||
| `ProbeInterface` is a library that defines a standard for specifying electrode layouts on probes. | ||
| The `ProbeInterface` library includes layouts for many common probes. | ||
| [ProbeInterface](https://github.com/SpikeInterface/probeinterface) is a standard for specifying electrode layouts on probes. |
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.
that page says that it is "Python package to handle probe layout, geometry and wiring to device." .
I doubt that it is a standard per se even if might be the only place where such defined.
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.
It's both. ProbeInterface defines the probe interface json schema, which makes it a standard
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.
we might have different understanding on what "standard means ;) but ok, we can discuss further in the main PR etc.
|
|
||
| Probes are physical devices used for recording microephys data. | ||
| They can be permanently implanted (chronic recordings) or inserted temporarily for the recording (acute recordings). | ||
| Probes are electrode-bearing devices that interface with neural tissue to record electrophysiological activity, ranging from multi-electrode arrays to single recording pipettes. They can be permanently implanted (chronic recordings) or inserted temporarily for the recording (acute recordings). |
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.
note: although I do not think it is adopted wide practice yet, it is better to stick to one sentence per line -- minimizes patches etc.
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.
@poldrack even now created https://pypi.org/project/mdnewline/
src/modality-specific-files/microelectrode-electrophysiology.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Yaroslav Halchenko <[email protected]>
see child issue: SpikeInterface/probeinterface_library#21