search

JuliaDynamics / TransferEntropy.jl

Transfer entropy (conditional mutual information) estimators for the Julia language
https://juliadynamics.github.io/CausalityTools.jl/stable/
Other
14 stars 5 forks source link

Convienience method for conditional mutual information #86

Closed kahaaga closed 2 years ago

codecov-commenter commented 2 years ago

Codecov Report

Merging #86 (973db2a) into master (f5aa11c) will decrease coverage by 0.24%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   79.36%   79.12%   -0.25%     
==========================================
  Files          10       10              
  Lines         475      479       +4     
==========================================
+ Hits          377      379       +2     
- Misses         98      100       +2
Impacted Files Coverage Δ
src/mutualinfo/interface.jl 87.50% <100.00%> (+4.16%) :arrow_up:
src/mutualinfo/nearestneighbor.jl 100.00% <100.00%> (ø)
src/transferentropy/autoutils.jl 95.27% <0.00%> (-1.58%) :arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Datseris commented 2 years ago

I would recommend to err on the side of clarity instead of brevity, and hence use a name like conditional_mutualinfo. Other than that, LGTM!