NDI-matlab functionality: reading from ndi.element.timeseries objects, writing ndi.neuron objects#157
Open
stevevanhooser wants to merge 23 commits intoJaneliaSciComp:masterfrom
Open
NDI-matlab functionality: reading from ndi.element.timeseries objects, writing ndi.neuron objects#157stevevanhooser wants to merge 23 commits intoJaneliaSciComp:masterfrom
stevevanhooser wants to merge 23 commits intoJaneliaSciComp:masterfrom
Conversation
prevents overwriting of prm parameter values by default values fixes renumbering of spike sites when KiloSort returns partial channel map
Collaborator
|
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. |
Author
|
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 |
Collaborator
|
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.