Conversation
…nto NWB_updated
|
@apdavison @legouee I think we can merge this PR soon to have a first version of the NWBIO in the next release. The most important change requests for this would be the incorporation of some testfiles also on gin (if you have some, as we can then tag the release to a fixed version of the test files), the incorporation of the BaseTests and some completion that is still lacking on the docstrings. Also I am not sure about some of the discussions in this thread form the end of last year, if there are outdated, could you resolve them? |
Co-authored-by: Ben Dichter <ben.dichter@gmail.com>
Co-authored-by: Andrew Davison <andrew.davison@cnrs.fr>
Implement suggestions from review of PR for nwb2 branch
|
@legouee @apdavison The test file is now on gin. Can you include it in the set of files used by the unittests and also include the common rawio tests by inheriting from |
|
@JuliaSprenger : since it now use |
|
There are some warning in test_nwb.py |
|
@apdavison @legouee I cleaned all open comments, moved unresolved ones to #1013 for the future. I fixed the layout to be pep8 compatible and fixed some docstrings / typos. Under the assumption that you don't plan to include apdavison#6 here any more I would be ok to merge the current version and fix the remaining issues listed in #1013 in the next release. |
This PR adds NWBIO, an IO module for reading and writing the Neurodata Without Borders format (cf #221).
This is a work in progress, which doesn't support all possible contents of an NWB file, but I think it is good enough for people to try it, and let us know (by commenting here) what doesn't work, or could be done better. Many thanks to @legouee for her work on this.
TODO
Usage example: https://github.com/apdavison/python-neo/blob/nwb2/examples/NWB-Allen-Institute-Example.ipynb