-
Notifications
You must be signed in to change notification settings - Fork 229
New Gibbs sampler using condition
#2099
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 66 commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
b3db449
initial work on the new gibbs sampler
torfjelde efe15db
added tests for the new Gibbs sampler
torfjelde 35f9f15
added tests for new Gibbs
torfjelde 072cf6b
new Gibbs is now sampling (correctly) sequentially
torfjelde 2f25199
let's not overload merge just yet
torfjelde 7ca26bb
export GibbsV2 + added more samplers to the tests
torfjelde 8b0de21
added TODO comment
torfjelde 90fd854
Merge branch 'master' into torfjelde/new-gibbs
yebai b960418
Merge remote-tracking branch 'origin/torfjelde/new-gibbs' into torfje…
torfjelde d000382
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 144b9c0
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 014fbe2
removed lots of varinfo related merging functionality that is now
torfjelde 63d64e6
shifting some code around
torfjelde 0dcd5bf
removed redundant constructor for GibbsV2
torfjelde c29efc1
added GibbsContext which is similar to FixContext but also computes
torfjelde bff0786
adopted the rerun mechanism in Gibbs for GibbsV2, thus fixing the
torfjelde 5955167
broken tests are no longer broken
torfjelde 614dc52
fix issues with dot_tilde_* impls for GibbsContext
torfjelde 7776f82
fix for dot_tilde_assume when using GibbsContext
torfjelde c65b7e9
fixed re-running of models for Gibbs sampling properly this time
torfjelde d1e94e7
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 16ddca2
added new gibbs to tests
torfjelde ba8c6e1
added some further comments on why we need `GibbsContext`
torfjelde 53bd707
went back to using `DynamicPPL.condition` rather than using custom
torfjelde b38a82a
add concrete comment about reverting changes for `gibbs_condition`
torfjelde adc67be
Update test/mcmc/gibbs_new.jl
torfjelde 3b5a74c
fixed recursive definition of `condition` varinfos
torfjelde 0578c35
Merge remote-tracking branch 'origin/torfjelde/new-gibbs' into torfje…
torfjelde f87e2d1
use `fix` instead of `condition`
torfjelde 1c1d9b7
Revert "use `fix` instead of `condition`"
torfjelde 6be7ab9
rmeoved unnused symbol
torfjelde a143cc4
Revert "went back to using `DynamicPPL.condition` rather than using c…
torfjelde 4d37f5f
bump compat entry of DynamicPPL so we can overload acclogp!
torfjelde f39e636
update assume for SMC samplers to make use of new `acclogp!`
torfjelde 9cc5397
added proper impl of acclogp!! for SMC samplers + made accessing
torfjelde c93944c
added experimental module and moved gibbs to it
torfjelde e6ce62f
fixed now-inccorect references in new gibbs file
torfjelde 475e264
updated gibbs tests
torfjelde 463f31c
moved experimental gibbs tests
torfjelde a79ad92
updated tests to include experiemntal tests
torfjelde b2a9566
removed refrences to previews tests of experimental Gibbs sampler
torfjelde ead7535
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 1266639
removed solved TODO
torfjelde 7bd0831
added a comment on `reconstruct_getvalue` usage
torfjelde c74727f
bump patch version
torfjelde 8b46f11
added comments on future work
torfjelde a81cf09
Update test/experimental/gibbs.jl
torfjelde f280caa
fixed bug where particle samplers didn't properly account for
torfjelde 86b859c
Merge remote-tracking branch 'origin/torfjelde/new-gibbs' into torfje…
torfjelde b1dcadf
relax atol for a numerical test with Gibbs a bit
torfjelde 0f0bfd5
Merge branch 'master' into torfjelde/new-gibbs
yebai 5e047ef
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 4bde75a
fixed bug with `AbstractDict` constructor for experimental `Gibbs`
torfjelde 90dcb83
Merge branch 'master' into torfjelde/new-gibbs
torfjelde b54e6eb
aaaalways link the varinfo in the new Gibbs sampler, just to be sure
torfjelde e7ad682
add test to cover recent improvement to `DynamicPPL.subset`
torfjelde 2713316
bump compat entry for DynamicPPL
torfjelde fb29556
added some docstrings
torfjelde d3a13ad
fixed test
torfjelde 81bf9c0
fixed import
torfjelde d340e58
another attempt at fixing tests
torfjelde 6f6fe7a
another attempt at fixing tests
torfjelde 9c93162
attempt at fix tests
torfjelde cfa8fb3
forgot something in previos commit
torfjelde 15e83b9
cleaned up the experimental Gibbs sampler a bit
torfjelde aed307a
removed accidentaly psuedocode inclusion
torfjelde 04c177a
Apply suggestions from code review
torfjelde 0acd27b
relaxed olerance in one MH test a bit
torfjelde 186d2e9
Merge remote-tracking branch 'origin/torfjelde/new-gibbs' into torfje…
torfjelde 74b8e8c
Merge branch 'master' into torfjelde/new-gibbs
torfjelde 0f30514
bump patch version
torfjelde File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
module Experimental | ||
|
||
using Random: Random | ||
using AbstractMCMC: AbstractMCMC | ||
using DynamicPPL: DynamicPPL, VarName | ||
using Setfield: Setfield | ||
|
||
using DocStringExtensions: TYPEDFIELDS | ||
using Distributions | ||
|
||
using ..Turing: Turing | ||
using ..Turing.Inference: gibbs_rerun, InferenceAlgorithm | ||
|
||
include("gibbs.jl") | ||
|
||
end |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.