SilverLabUCL / PySilverLabNWB

Python tools for working with Silver Lab data in the NWB2 format
MIT License
1 stars 0 forks source link

Import code from old repository #4

Closed jonc125 closed 4 years ago

jonc125 commented 5 years ago

Fixes #2. See also https://github.com/SilverLabUCL/AnalysisPipeline/pull/41 on legacy repository.

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0b19abe). Click here to learn what that means. The diff coverage is 26.35%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #4   +/-   ##
=========================================
  Coverage          ?   26.41%           
=========================================
  Files             ?       11           
  Lines             ?     1435           
  Branches          ?      234           
=========================================
  Hits              ?      379           
  Misses            ?     1038           
  Partials          ?       18
Impacted Files Coverage Δ
src/silverlabnwb/metadata_gui.py 0% <0%> (ø)
src/silverlabnwb/nwb_util.py 0% <0%> (ø)
src/silverlabnwb/subsample_nwb.py 0% <0%> (ø)
src/silverlabnwb/script.py 0% <0%> (ø)
tests/conftest.py 100% <100%> (ø)
src/silverlabnwb/__init__.py 100% <100%> (ø)
src/silverlabnwb/signature.py 17.2% <17.2%> (ø)
src/silverlabnwb/nwb_file.py 40.71% <40.71%> (ø)
tests/test_labview_import.py 43.75% <43.75%> (ø)
tests/test_epochs.py 85.04% <85.04%> (ø)
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0b19abe...90f5ed2. Read the comment docs.

jonc125 commented 5 years ago

The test failures now we're testing the import process are expected, as there are still some outstanding pynwb issues. I think this is good to review and merge as a base point though @ageorgou

jonc125 commented 4 years ago

There are still some issues, but I think they'll be resolved by #13, so merging this anyway.