Skip to content
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

New version: Turing v0.30.9 #105315

Merged
merged 1 commit into from
Apr 21, 2024

Conversation

JuliaRegistrator
Copy link
Contributor

## New features

- `Turing.Experimental` module where more experimental features will go, with the aim of eventually making its way into `Turing` proper.
- `Turing.Experimental.Gibbs` is a new implementation of the `Gibbs` sampler which provides much greater flexibility specifying which variables should use which sampler. Note that this is in the `Turing.Experimental` module, and thus will be prone to changes.

UUID: fce5fe82-541a-59a6-adf8-730c64b5f9a0
Repo: https://github.com/TuringLang/Turing.jl.git
Tree: 4170ff68a0aa2b26b3944ba7bd8789a46d34e6bc

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
JuliaRegistrator referenced this pull request in TuringLang/Turing.jl Apr 21, 2024
* initial work on the new gibbs sampler

* added tests for the new Gibbs sampler

* added tests for new Gibbs

* new Gibbs is now sampling (correctly) sequentially

* let's not overload merge just yet

* export GibbsV2 + added more samplers to the tests

* added TODO comment

* removed lots of varinfo related merging functionality that is now
available in DynamicPPL

* shifting some code around

* removed redundant constructor for GibbsV2

* added GibbsContext which is similar to FixContext but also computes
the log-prob of the fixed variables

* adopted the rerun mechanism in Gibbs for GibbsV2, thus fixing the
issues with some of the tests for GibbsV2

* broken tests are no longer broken

* fix issues with dot_tilde_* impls for GibbsContext

* fix for dot_tilde_assume when using GibbsContext

* fixed re-running of models for Gibbs sampling properly this time

* added new gibbs to tests

* added some further comments on why we need `GibbsContext`

* went back to using `DynamicPPL.condition` rather than using custom
`GibbsContext` while we wait for
TuringLang/DynamicPPL.jl#563 to be merged

* add concrete comment about reverting changes for `gibbs_condition`

* Update test/mcmc/gibbs_new.jl

Co-authored-by: Hong Ge <[email protected]>

* fixed recursive definition of `condition` varinfos

* use `fix` instead of `condition`

* Revert "use `fix` instead of `condition`"

This reverts commit f87e2d1.

* rmeoved unnused symbol

* Revert "went back to using `DynamicPPL.condition` rather than using custom"

This reverts commit 53bd707.

* bump compat entry of DynamicPPL so we can overload acclogp!

* update assume for SMC samplers to make use of new `acclogp!`

* added proper impl of acclogp!! for SMC samplers + made accessing
task local varinfo and rng a bit nicer

* added experimental module and moved gibbs to it

* fixed now-inccorect references in new gibbs file

* updated gibbs tests

* moved experimental gibbs tests

* updated tests to include experiemntal tests

* removed refrences to previews tests of experimental Gibbs sampler

* removed solved TODO

* added a comment on `reconstruct_getvalue` usage

* bump patch version

* added comments on future work

* Update test/experimental/gibbs.jl

* fixed bug where particle samplers didn't properly account for
weightings of logpdf, etc.

* relax atol for a numerical test with Gibbs a bit

* fixed bug with `AbstractDict` constructor for experimental `Gibbs`

* aaaalways link the varinfo in the new Gibbs sampler, just to be sure

* add test to cover recent improvement to `DynamicPPL.subset`

ref: TuringLang/DynamicPPL.jl#587

* bump compat entry for DynamicPPL

* added some docstrings

* fixed test

* fixed import

* another attempt at fixing tests

* another attempt at fixing tests

* attempt at fix tests

* forgot something in previos commit

* cleaned up the experimental Gibbs sampler a bit

* removed accidentaly psuedocode inclusion

* Apply suggestions from code review

* relaxed olerance in one MH test a bit

* bump patch version

---------

Co-authored-by: Hong Ge <[email protected]>
Copy link
Contributor

Your new version pull request met all of the guidelines for auto-merging and is scheduled to be merged in the next round.


If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment. You can edit blocking comments, adding [noblock] to them in order to unblock auto-merging.

@JuliaTagBot JuliaTagBot merged commit de11a14 into master Apr 21, 2024
17 checks passed
@JuliaTagBot JuliaTagBot deleted the registrator-turing-fce5fe82-v0.30.9-1deb94ed27 branch April 21, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants