Closed h-mayorquin closed 2 years ago
@h-mayorquin this looks really good, thanks for working on it! Could you also add this to the docs in this PR?
@h-mayorquin this looks really good, thanks for working on it! Could you also add this to the docs in this PR?
@bendichter , Done in the last commit.
Merging #605 (7a9be7a) into main (8a78f80) will increase coverage by
0.01%
. The diff coverage is90.32%
.
@@ Coverage Diff @@
## main #605 +/- ##
==========================================
+ Coverage 88.42% 88.44% +0.01%
==========================================
Files 59 60 +1
Lines 3222 3253 +31
==========================================
+ Hits 2849 2877 +28
- Misses 373 376 +3
Flag | Coverage Δ | |
---|---|---|
unittests | 88.44% <90.32%> (+0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
...ces/behavior/deeplabcut/deeplabcutdatainterface.py | 90.00% <90.00%> (ø) |
|
...rc/nwb_conversion_tools/datainterfaces/__init__.py | 100.00% <100.00%> (ø) |
I will move this to the new repo.
I will close this as the PR in the new repo is the one were the latest updates have been pushed and is now mostly ready.
This a re-hash of #414 and should close #414.
I changed the
run_conversion
signature to the new style.