-
Notifications
You must be signed in to change notification settings - Fork 22
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
NDI-matlab functionality: reading from ndi.element.timeseries objects, writing ndi.neuron objects #157
base: master
Are you sure you want to change the base?
Conversation
prevents overwriting of prm parameter values by default values fixes renumbering of spike sites when KiloSort returns partial channel map
You're right about the export documentation, there's not much (or none) to speak of. I can work on standing that up. Can I ask that you add a few unit tests? I'm happy to talk offline about the test framework I've set up. |
Hi Alan - Yes, I'd be delighted to add unit tests. Can you point me to an example, maybe an Intan example, or any other example that you think would be useful? -Steve |
Unit tests are another thing I'm still building out -- in particular there aren't any reading or import/export tests (you probably saw this already). I think at a bare minimum, if you can write something out with NDI and read it back in with JRCLUST, and everything is where you expect it to be, that would be useful. We can talk about edge cases when we meet. |
These modifications allow:
Requires http://github.com/VH-Lab/NDI-matlab for these functions (error is given if not installed)
Still needed:
There appears to be no documentation for the export methods (or, I didn't find it). I would like to add documentation to the export ndi function in the documentation.