DistributionsAD.jl icon indicating copy to clipboard operation
DistributionsAD.jl copied to clipboard

Low test coverage

Open mohdibntarek opened this issue 5 years ago • 0 comments

Currently the test coverage is pretty low. Some of the functions defined here are used in packages downstream so we get indirect testing, but this is not good enough for CI. I went through the coverage report and found the following missing tests:

  • unsafe_cholesky
  • For
  • TuringWishart and TuringInverseWishart
    • insupport
    • size
    • rank
    • mean
    • mode
    • meanlogdet
    • entropy
    • cov
    • var
    • rand
    • Constructor with df::TrackedReal and S::Matrix
  • TuringDenseMvNormal
    • length
    • rand
    • Matrix logpdf
  • TuringDiagMvNormal
    • params
    • dim
    • size
    • rand
  • TuringScalMvNormal
    • length
    • size
    • Single and multi rand
  • TuringMvLogNormal
    • length
    • Single and multi rand
    • Vector and matrix logpdf out of support
    • Constructor with m::TrackedVector or m::AbstractVector and A::UniformScaling
  • TuringUniform
    • No arg constructor
    • logpdf of Distributions.Uniform with x::TrackedReal
  • Semicircle
    • logpdf of Distributions.Semicircle with x::TrackedReal
  • inv of pi
  • NegativeBinomial
    • nbinomlogpdf - not sure why because we are testing NegativeBinomial and this should be called
  • TuringPoissonBinomial
    • quantile
    • minimum

Let's make the coverage 100%!

mohdibntarek avatar Feb 16 '20 10:02 mohdibntarek