search

stripe / rainier

Bayesian inference in Scala.
https://rainier.fit
Apache License 2.0
432 stars 51 forks source link

Rework Sampler interface #451

Closed avibryant closed 4 years ago

avibryant commented 4 years ago

Move iterations and warmupIterations into Sampler and out of sample. Move thin into a post-processing step on Sample.

codecov-io commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (0.3-dev@a8859a8). Click here to learn what that means. The diff coverage is 39.13%.

Impacted file tree graph

@@            Coverage Diff             @@
##             0.3-dev     #451   +/-   ##
==========================================
  Coverage           ?   51.31%           
==========================================
  Files              ?       73           
  Lines              ?     2900           
  Branches           ?      161           
==========================================
  Hits               ?     1488           
  Misses             ?     1412           
  Partials           ?        0
Impacted Files Coverage Δ
...ain/scala/com/stripe/rainier/sampler/Sampler.scala 0% <ø> (ø)
...c/main/scala/com/stripe/rainier/sampler/Ehmc.scala 0% <0%> (ø)
...rc/main/scala/com/stripe/rainier/core/Sample.scala 40% <0%> (ø)
...e/src/main/scala/com/stripe/rainier/core/SBC.scala 13.86% <0%> (ø)
...rc/main/scala/com/stripe/rainier/sampler/HMC.scala 92.85% <100%> (ø)
.../main/scala/com/stripe/rainier/core/Discrete.scala 73.39% <100%> (ø)
.../main/scala/com/stripe/rainier/core/SBCModel.scala 74.5% <66.66%> (ø)
...src/main/scala/com/stripe/rainier/core/Model.scala 63.82% <80%> (ø)

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 a8859a8...6bd196b. Read the comment docs.