G-Node / nix

Neuroscience information exchange format
https://readthedocs.org/projects/nixio/
Other
67 stars 36 forks source link

support DataFrames in features #829

Open jgrewe opened 3 years ago

jgrewe commented 3 years ago

unifinished!

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 96ebeb4bfd960809f3fde8948770cbbec3f7d65c into 1875eecf0d875dd16200d49c7fd5f8b631268ed3 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging c2b10bb563ec0b9bb71dc82b06fa27a778161ce2 into ec18e568c0f42ffb58673431931b211b47084d0d - view on LGTM.com

new alerts:

codecov[bot] commented 3 years ago

Codecov Report

Merging #829 (c0f4300) into master (e6802d4) will decrease coverage by 0.10%. The diff coverage is 90.00%.

:exclamation: Current head c0f4300 differs from pull request most recent head ef78e15. Consider uploading reports for the commit ef78e15 to get more accurate results

@@            Coverage Diff             @@
##           master     #829      +/-   ##
==========================================
- Coverage   95.70%   95.59%   -0.11%     
==========================================
  Files         201      201              
  Lines       13657    13838     +181     
==========================================
+ Hits        13070    13229     +159     
- Misses        587      609      +22     
Impacted Files Coverage Δ
include/nix/MultiTag.hpp 100.00% <ø> (ø)
include/nix/Tag.hpp 100.00% <ø> (ø)
include/nix/base/IBaseTag.hpp 100.00% <ø> (ø)
include/nix/base/IFeature.hpp 100.00% <ø> (ø)
test/BaseTestFeature.hpp 100.00% <ø> (ø)
src/MultiTag.cpp 79.59% <50.00%> (-4.13%) :arrow_down:
src/Feature.cpp 78.12% <64.70%> (-16.32%) :arrow_down:
src/util/dataAccess.cpp 89.27% <66.66%> (-0.31%) :arrow_down:
include/nix/Exception.hpp 91.52% <75.00%> (+0.45%) :arrow_up:
src/Tag.cpp 89.15% <88.88%> (-0.18%) :arrow_down:
... and 9 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 e6802d4...ef78e15. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts and fixes 1 when merging 663b23785aaa4da35948b45b7f3b0d86b14fc607 into c865fd01f2728f219bc332674aa970842e916e25 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 1 when merging ef78e15671af140870835a016f540165ada7a5fa into e6802d4867fdff8eafd2e2f1d309a3750c3a74ec - view on LGTM.com

new alerts:

fixed alerts: