JuliaRobotics / KernelDensityEstimate.jl

Kernel Density Estimate with product approximation using multiscale Gibbs sampling
GNU Lesser General Public License v2.1
23 stars 7 forks source link

Feature/hybridmanifolds #39

Closed dehann closed 5 years ago

dehann commented 5 years ago

Will test hybrid manifolds after this PR is merged

dehann commented 5 years ago

This code is only tested for either Euclidean of single Circular dimension per kde!

codecov-io commented 5 years ago

Codecov Report

Merging #39 into master will increase coverage by 1.52%. The diff coverage is 79.71%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #39      +/-   ##
=========================================
+ Coverage   71.58%   73.1%   +1.52%     
=========================================
  Files           8       9       +1     
  Lines         929     963      +34     
=========================================
+ Hits          665     704      +39     
+ Misses        264     259       -5
Impacted Files Coverage Δ
src/KernelDensityEstimate.jl 25% <0%> (-75%) :arrow_down:
src/KDE01.jl 92.4% <100%> (+1.78%) :arrow_up:
src/MSGibbs01.jl 81.11% <100%> (+0.41%) :arrow_up:
src/BallTreeDensity01.jl 51.36% <50%> (ø) :arrow_up:
src/CrossValidation.jl 70.9% <70.9%> (ø)
src/DualTree01.jl 71.93% <78.88%> (+0.99%) :arrow_up:
src/BallTree01.jl 77.71% <96%> (+5.25%) :arrow_up:

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 4172f54...5d55294. Read the comment docs.

dehann commented 5 years ago

global flag for direct evaluation of kernels vs computational geometry (previously designed for Euclidean manifold only)