Update for proposed DynamicPPL 0.32 (#2419)

* Add DynamicPPL integration tests

* Bump DynamicPPL compat -> 0.32

* Remove / export deprecated functions as needed

Including @SubModel, to_submodel, link!, invlink!

* Update submodel names as per DPPL 0.32

* Enable concurrency on PR testing to avoid wastage

* Bump Turing patch version

* Improve descriptions of tests

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

* Update test/dynamicppl/compiler.jl

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: Hong Ge <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Author: 3 people

.github/workflows/Tests.yml

@@ -6,6 +6,11 @@ on:
       - master
   pull_request:
 
+# Cancel existing tests on the same PR if a new commit is added to a pull request
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref || github.run_id }}
+  cancel-in-progress: ${{ startsWith(github.ref, 'refs/pull/') }}
+
 jobs:
   test:
     # Use matrix.test.name here to avoid it taking up the entire window width

Project.toml

@@ -1,6 +1,6 @@
 name = "Turing"
 uuid = "fce5fe82-541a-59a6-adf8-730c64b5f9a0"
-version = "0.35.4"
+version = "0.35.5"
 
 [deps]
 ADTypes = "47edcb42-4c32-4615-8424-f2b9edc5f35b"
@@ -63,7 +63,7 @@ Distributions = "0.23.3, 0.24, 0.25"
 DistributionsAD = "0.6"
 DocStringExtensions = "0.8, 0.9"
 DynamicHMC = "3.4"
-DynamicPPL = "0.29, 0.30.4, 0.31"
+DynamicPPL = "0.32"
 EllipticalSliceSampling = "0.5, 1, 2"
 ForwardDiff = "0.10.3"
 Libtask = "0.8.8"

docs/src/api.md

@@ -37,11 +37,11 @@ even though [`Prior()`](@ref) is actually defined in the `Turing.Inference` modu
 
 ### Modelling
 
-| Exported symbol | Documentation                     | Description                                  |
-|:--------------- |:--------------------------------- |:-------------------------------------------- |
-| `@model`        | [`DynamicPPL.@model`](@extref)    | Define a probabilistic model                 |
-| `@varname`      | [`AbstractPPL.@varname`](@extref) | Generate a `VarName` from a Julia expression |
-| `@submodel`     | [`DynamicPPL.@submodel`](@extref) | Define a submodel                            |
+| Exported symbol | Documentation                       | Description                                  |
+|:--------------- |:----------------------------------- |:-------------------------------------------- |
+| `@model`        | [`DynamicPPL.@model`](@extref)      | Define a probabilistic model                 |
+| `@varname`      | [`AbstractPPL.@varname`](@extref)   | Generate a `VarName` from a Julia expression |
+| `to_submodel`   | [`DynamicPPL.to_submodel`](@extref) | Define a submodel                            |
 
 ### Inference
 

src/Turing.jl

@@ -71,15 +71,17 @@ using DynamicPPL:
     decondition,
     fix,
     unfix,
-    conditioned
+    conditioned,
+    to_submodel
 using StatsBase: predict
 using Bijectors: ordered
 using OrderedCollections: OrderedDict
 
 # Turing essentials - modelling macros and inference algorithms
 export @model,                 # modelling
     @varname,
-    @submodel,
+    @submodel,  # Deprecated
+    to_submodel,
     DynamicPPL,
     Prior,                  # Sampling from the prior
     MH,                     # classic sampling

src/mcmc/Inference.jl

@@ -6,8 +6,6 @@ using DynamicPPL:
     VarInfo,
     TypedVarInfo,
     islinked,
-    invlink!,
-    link!,
     setindex!!,
     push!!,
     setlogp!!,

test/Project.toml

@@ -1,9 +1,11 @@
 [deps]
 AbstractMCMC = "80f14c24-f653-4e6a-9b94-39d6b0f70001"
+AbstractPPL = "7a57a42e-76ec-4ea3-a279-07e840d6d9cf"
 AdvancedMH = "5b7e9947-ddc0-4b3f-9b55-0d8042f74170"
 AdvancedPS = "576499cb-2369-40b2-a588-c64705576edc"
 AdvancedVI = "b5ca4192-6429-45e5-a2d9-87aec30a685c"
 Aqua = "4c88cf16-eb10-579e-8560-4a9242c79595"
+BangBang = "198e06fe-97b7-11e9-32a5-e1d131e6ad66"
 Clustering = "aaaa29a8-35af-508c-8bc3-b662a17a0fe5"
 Distributions = "31c24e10-a181-5473-b8eb-7969acd0382f"
 DistributionsAD = "ced4e74d-a319-5a8a-b0ac-84af2272839c"
@@ -45,7 +47,7 @@ Clustering = "0.14, 0.15"
 Distributions = "0.25"
 DistributionsAD = "0.6.3"
 DynamicHMC = "2.1.6, 3.0"
-DynamicPPL = "0.29, 0.30.4, 0.31"
+DynamicPPL = "0.32.2"
 FiniteDifferences = "0.10.8, 0.11, 0.12"
 ForwardDiff = "0.10.12 - 0.10.32, 0.10"
 HypothesisTests = "0.11"