search

catalystneuro / nwb-conversion-tools

Create NWB files by converting and combining neural data in proprietary formats and adding essential metadata.
https://nwb-conversion-tools.readthedocs.io/en/main/
BSD 3-Clause "New" or "Revised" License
25 stars 12 forks source link

[Gallery] Pin SpikeInterface to dev branch #598

Closed CodyCBakerPhD closed 2 years ago

CodyCBakerPhD commented 2 years ago

@h-mayorquin I had to temporarily disable this doctest for the latest release.

This re-enables it at least on a dev branch with associated SI version pin. Keeping this PR here in draft until SI releases in case we need to do any hotfixes for latest release.

SpikeInterface is expected to issue next release at the end of July, at which point this can be re-introduced.

codecov[bot] commented 2 years ago

Codecov Report

Merging #598 (77fe60e) into main (72a56b0) will increase coverage by 0.77%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #598      +/-   ##
==========================================
+ Coverage   87.56%   88.34%   +0.77%     
==========================================
  Files          59       59              
  Lines        3218     3218              
==========================================
+ Hits         2818     2843      +25     
+ Misses        400      375      -25
Flag Coverage Δ
unittests 88.34% <ø> (+0.77%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/nwb_conversion_tools/tools/data_transfers.py 91.54% <0.00%> (+35.21%) :arrow_up:
h-mayorquin commented 2 years ago

This was transfered to https://github.com/catalystneuro/neuroconv/pull/26 Closing it here