nathanaelbosch / ProbNumDiffEq.jl

Probabilistic Numerical Differential Equation solvers via Bayesian filtering and smoothing
MIT License
119 stars 16 forks source link

Make ClassicSovlerInit use backward transitions and completely remove `smooth!` from the package #262

Closed nathanaelbosch closed 1 year ago

nathanaelbosch commented 1 year ago

Solves #259

Maybe do some benchmarks again before merging, just to see if/how this influenced performance

codecov[bot] commented 1 year ago

Codecov Report

Merging #262 (ba7d92f) into main (e627234) will decrease coverage by 0.36%. Report is 4 commits behind head on main. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #262      +/-   ##
==========================================
- Coverage   95.75%   95.40%   -0.36%     
==========================================
  Files          35       35              
  Lines        1462     1435      -27     
==========================================
- Hits         1400     1369      -31     
- Misses         62       66       +4     
Files Coverage Δ
src/filtering/smooth.jl 100.00% <ø> (ø)
src/initialization/classicsolverinit.jl 97.40% <100.00%> (+0.10%) :arrow_up:

... and 3 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

nathanaelbosch commented 1 year ago

Held up by Fenrir.jl's patch release with increased ProbNumDiffEq.jl compat: https://github.com/JuliaRegistries/General/pull/94415