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

Add spikeinterface support to `PhySortingInterface` #567

Closed h-mayorquin closed 2 years ago

h-mayorquin commented 2 years ago

As stated in the title. Linked to https://github.com/catalystneuro/neuroconv/issues/60.

CodyCBakerPhD commented 2 years ago

Nice and simple!

codecov[bot] commented 2 years ago

Codecov Report

Merging #567 (8e22be1) into main (0a3c829) will increase coverage by 0.21%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     catalystneuro/nwb-conversion-tools#567      +/-   ##
==========================================
+ Coverage   88.07%   88.28%   +0.21%     
==========================================
  Files          59       59              
  Lines        3135     3141       +6     
==========================================
+ Hits         2761     2773      +12     
+ Misses        374      368       -6     
Flag Coverage Δ
unittests 88.28% <100.00%> (+0.21%) :arrow_up:

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

Impacted Files Coverage Δ
...rfaces/ecephys/blackrock/blackrockdatainterface.py 92.10% <100.00%> (+0.32%) :arrow_up:
...ols/datainterfaces/ecephys/phy/phydatainterface.py 100.00% <100.00%> (ø)
...rsion_tools/tools/spikeinterface/spikeinterface.py 85.32% <0.00%> (+1.15%) :arrow_up: