openep / openep-core

Core OpenEP code - Matlab implementation
Apache License 2.0
9 stars 6 forks source link

Batch anonymising and converting of datasets #63

Closed p-j-smith closed 2 years ago

p-j-smith commented 2 years ago

Changes made:

codecov[bot] commented 2 years ago

Codecov Report

Merging #63 (3a613ba) into develop (ea779f7) will increase coverage by 1.10%. The diff coverage is 56.14%.

@@            Coverage Diff             @@
##           develop     #63      +/-   ##
==========================================
+ Coverage     7.03%   8.14%   +1.10%     
==========================================
  Files          144     147       +3     
  Lines         6805    6862      +57     
==========================================
+ Hits           479     559      +80     
+ Misses        6326    6303      -23     
Impacted Files Coverage Δ
editUserdata.m 0.00% <0.00%> (ø)
generateInterpData.m 0.00% <0.00%> (ø)
getAblationArea.m 0.00% <0.00%> (ø)
getAnatomicalStructures.m 0.00% <0.00%> (ø)
getArea.m 0.00% <0.00%> (ø)
getClosedSurface.m 0.00% <0.00%> (ø)
getConductionVelocity.m 0.00% <0.00%> (ø)
getEarliestActivationSite.m 0.00% <0.00%> (ø)
getFaces.m 0.00% <0.00%> (ø)
getLatestActivationSite.m 0.00% <0.00%> (ø)
... and 10 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 43d91a0...3a613ba. Read the comment docs.

drsewilliams commented 2 years ago

@p-j-smith , I will approve this for merging now; but please can you add the changes to the changelog.md file (*which only exists in its own branch just now) (I have asked @nicklinton to approve the PR related to the changelog branch such that it can be propagated into the other feature branches too).