search

JuliaGaussianProcesses / Stheno.jl

Probabilistic Programming with Gaussian processes in Julia
Other
338 stars 26 forks source link

Allow AbstractGPs in WrappedGP #217

Closed willtebbutt closed 2 years ago

willtebbutt commented 2 years ago

This is a bug fix.

Currently, WrappedGPs only permit GPs to be wrapped, which is limiting.

An unrelated change is the removal of a lot of block-diagonal code that was redundant and broken.

codecov[bot] commented 2 years ago

Codecov Report

Merging #217 (270188d) into master (999cd2c) will increase coverage by 3.10%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
+ Coverage   86.42%   89.53%   +3.10%     
==========================================
  Files          17       16       -1     
  Lines         442      344      -98     
==========================================
- Hits          382      308      -74     
+ Misses         60       36      -24
Impacted Files Coverage Δ
src/gp/gp.jl 100.00% <100.00%> (ø)
src/util/proper_type_piracy.jl 0.00% <0.00%> (-100.00%) :arrow_down:
src/util/block_arrays/dense.jl 70.58% <0.00%> (-29.42%) :arrow_down:
src/composite/composite_gp.jl 100.00% <0.00%> (ø)
src/abstract_gp.jl 87.50% <0.00%> (+1.78%) :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 999cd2c...270188d. Read the comment docs.