Skip to content

Commit

Permalink
Add suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Tor Erlend Fjelde <[email protected]>
  • Loading branch information
mhauru and torfjelde committed Nov 29, 2024
1 parent 38ac128 commit 7d6a983
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/mcmc/gibbs.jl
Original file line number Diff line number Diff line change
Expand Up @@ -509,7 +509,7 @@ function setparams_varinfo!!(
# update its logprob. To do this, we have to call evaluate!! with the sampler, rather
# than just a context, because ESS is peculiar in how it uses LikelihoodContext for
# some variables and DefaultContext for others.
return last(DynamicPPL.evaluate!!(model, params, sampler))
return last(DynamicPPL.evaluate!!(model, params, SamplingContext(sampler)))
end

function setparams_varinfo!!(

Check warning on line 515 in src/mcmc/gibbs.jl

View check run for this annotation

Codecov / codecov/patch

src/mcmc/gibbs.jl#L515

Added line #L515 was not covered by tests
Expand Down

0 comments on commit 7d6a983

Please sign in to comment.