Skip to content

Conversation

devmotion
Copy link
Member

This PR adds relevant files from the Turing repo in a Turing/ subfolder, together with their history.

The files were extracted with git-filter-repo:

git clone [email protected]:TuringLang/Turing.jl.git
cd Turing
git filter-repo --path src/contrib/inference/AdvancedSMCExtensions.jl --path src/core/container.jl --path src/inference/AdvancedSMC.jl --path test/core/container.jl --path test/inference/AdvancedSMC.jl --path-rename '':Turing/
git remote add origin [email protected]:TuringLang/AdvancedPS.jl.git
git pull --allow-unrelated-histories origin master
git checkout -b Turing
git push -u origin Turing

Hopefully this simplifies reviews of PRs such as #11 (see, e.g., #11 (comment)) since it would be clearer what parts of the existing implementation in Turing were changed. Moreover, by keeping the history it might be easier to track down changes and understand the implementation better.

adscib and others added 30 commits April 29, 2016 13:34
- Dict --> Array
- push! : preallocate memory
- forkc: remove forkc for the particles with only one chilcren.
# Conflicts:
#	README.md
Moved some untested functions to a separate branch.
Turing now works with Julia 0.5
# Conflicts:
#	README.md
#	docs/source/index.rst
# Conflicts:
#	src/core/compiler.jl
#	test/assume.jl
#	test/clonetask.jl
#	test/noparam.jl
#	test/predict.jl
#	test/runtests.jl
New interface and Gibbs sampling
Unify Mamba.Chain and Turing.Chain.
# Conflicts:
#	src/samplers/hmc.jl
#	src/samplers/sampler.jl
#	test/gibbs.jl
devmotion and others added 23 commits February 18, 2020 14:10
* Update Turing to AbstractMCMC 0.4

* Update sampling interface guide

* Simplify resume_from handling in particle methods

* Actually pass the RNG

* Use cat instead of vcat to avoid mysterious type issues

* Revert hack

* Add error message

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

* minor fixes

* bump DAD compat version

* test fixes

* use ForwardDiff for non-AD tests

* Update Project.toml

Co-Authored-By: David Widmann <[email protected]>

* apply David's suggestions

* Update src/Turing.jl

Co-Authored-By: David Widmann <[email protected]>

* David's comment

* David's and Will's comments

* Update Project.toml

Co-Authored-By: David Widmann <[email protected]>

* Update Project.toml

Co-Authored-By: David Widmann <[email protected]>

* Update src/core/ad.jl

Co-Authored-By: David Widmann <[email protected]>

* Update src/core/ad.jl

Co-Authored-By: David Widmann <[email protected]>

* Update src/core/ad.jl

Co-Authored-By: David Widmann <[email protected]>

* indentation

Co-authored-by: David Widmann <[email protected]>
* Remove distribution wrappers (~> DynamicPPL)

* Remove asssume/observer fallback code (~> DynamicPPL)

* Update imports/exports/qualified names

* Remove unnecessary code from utils (is in DPPL now)

* fix rebase

* bump DPPL compat version

Co-authored-by: mohamed82008 <[email protected]>
Update Turing to AbstractMCMC 1.0, AdvancedMH 0.5, and DynamicPPL 0.6
* Use `propagate!` instead of `Libtask.consume`

* Fix tests
* Make Turing compatible with DynamicPPL 0.7

* Uncomment test that should be fixed upstream

* Updating variables in MH currently only works for `VarInfo`

* Use DynamicPPL 0.7.1
* Allow sampling from the prior

* Apply suggestions from code review

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

* Use `getlogp` instead of `getlp`

* Add seed and increase number of samples

Co-authored-by: Hong Ge <[email protected]>
* Fix particle filters with adaptive resampling and add documentation

Fixes TuringLang/DynamicPPL.jl#104

* Fix and extend tests of `ParticleContainer`

* Move logevidence tests from DynamicPPL

* Add more convenient constructors for Particle Gibbs

* Relax type annotations

* Check for approximate equality only

* Add docstring and reference
* Make GibbsComponent a simple trait (fix 1306)

* Simplify isgibbscomponent; implement some other suggestions

* Temporary fix for Distributions dependency
* Remove `Turing.DEBUG`

* Remove `_debug`

* Update advi.jl

Co-authored-by: Hong Ge <[email protected]>
Co-authored-by: Cameron Pfiffer <[email protected]>
@yebai yebai merged commit 248846c into master Nov 30, 2020
@delete-merged-branch delete-merged-branch bot deleted the Turing branch November 30, 2020 15:19
@yebai
Copy link
Member

yebai commented Nov 30, 2020

Thanks, @devmotion - this is very helpful!

@yebai
Copy link
Member

yebai commented Nov 30, 2020

@devmotion maybe we can rename these files in a new PR, so that changes in #11 can be recognised by Github?

@devmotion
Copy link
Member Author

Sure, I just wanted to keep this PR clean.

@devmotion
Copy link
Member Author

Maybe it would be good as well to generate the default package layout with an empty module in a separate PR first, e.g., with PkgTemplates?

@yebai
Copy link
Member

yebai commented Nov 30, 2020

Maybe it would be good as well to generate the default package layout with an empty module in a separate PR first, e.g., with PkgTemplates?

Sounds good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.