From 99ffbbe047b09774b020336b49e16a66fa8ee0f8 Mon Sep 17 00:00:00 2001 From: Penelope Yong Date: Tue, 17 Jun 2025 00:51:42 +0100 Subject: [PATCH 1/4] Remove dead code --- src/context_implementations.jl | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/context_implementations.jl b/src/context_implementations.jl index fdd2dbcee..a74347c04 100644 --- a/src/context_implementations.jl +++ b/src/context_implementations.jl @@ -1,19 +1,8 @@ using Distributions: UnivariateDistribution, MultivariateDistribution, MatrixDistribution, Distribution -const AMBIGUITY_MSG = - "Ambiguous `LHS .~ RHS` or `@. LHS ~ RHS` syntax. The broadcasting " * - "can either be column-wise following the convention of Distributions.jl or " * - "element-wise following Julia's general broadcasting semantics. Please make sure " * - "that the element type of `LHS` is not a supertype of the support type of " * - "`AbstractVector` to eliminate ambiguity." - alg_str(spl::Sampler) = string(nameof(typeof(spl.alg))) -# utility funcs for querying sampler information -require_gradient(spl::Sampler) = false -require_particles(spl::Sampler) = false - # Allows samplers, etc. to hook into the final logp accumulation in the tilde-pipeline. function acclogp_assume!!(context::AbstractContext, vi::AbstractVarInfo, logp) return acclogp_assume!!(NodeTrait(acclogp_assume!!, context), context, vi, logp) From 0684573742203a3d390ab3ff1799f452560b90d8 Mon Sep 17 00:00:00 2001 From: Penelope Yong Date: Tue, 17 Jun 2025 00:54:08 +0100 Subject: [PATCH 2/4] More dead code --- src/context_implementations.jl | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/context_implementations.jl b/src/context_implementations.jl index a74347c04..1ff03cd61 100644 --- a/src/context_implementations.jl +++ b/src/context_implementations.jl @@ -1,7 +1,4 @@ -using Distributions: - UnivariateDistribution, MultivariateDistribution, MatrixDistribution, Distribution - -alg_str(spl::Sampler) = string(nameof(typeof(spl.alg))) +using Distributions: Distribution # Allows samplers, etc. to hook into the final logp accumulation in the tilde-pipeline. function acclogp_assume!!(context::AbstractContext, vi::AbstractVarInfo, logp) From 8907ec4653276185198f10111008fc72bc5dd8b9 Mon Sep 17 00:00:00 2001 From: Penelope Yong Date: Tue, 17 Jun 2025 00:57:14 +0100 Subject: [PATCH 3/4] no need that too --- src/context_implementations.jl | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/context_implementations.jl b/src/context_implementations.jl index 1ff03cd61..3ee88149e 100644 --- a/src/context_implementations.jl +++ b/src/context_implementations.jl @@ -1,5 +1,3 @@ -using Distributions: Distribution - # Allows samplers, etc. to hook into the final logp accumulation in the tilde-pipeline. function acclogp_assume!!(context::AbstractContext, vi::AbstractVarInfo, logp) return acclogp_assume!!(NodeTrait(acclogp_assume!!, context), context, vi, logp) From 4c14d7b7e9c649d6f48a964049e86098b276dcc5 Mon Sep 17 00:00:00 2001 From: Penelope Yong Date: Thu, 19 Jun 2025 11:03:44 +0100 Subject: [PATCH 4/4] bump patch --- HISTORY.md | 5 +++++ Project.toml | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/HISTORY.md b/HISTORY.md index a0f91a494..f216df783 100644 --- a/HISTORY.md +++ b/HISTORY.md @@ -1,5 +1,10 @@ # DynamicPPL Changelog +## 0.36.12 + +Removed several unexported functions. +The only notable one is `DynamicPPL.alg_str`, which was used in old versions of AdvancedVI and the Turing test suite. + ## 0.36.11 Make `ThreadSafeVarInfo` hold a total of `Threads.nthreads() * 2` logp values, instead of just `Threads.nthreads()`. diff --git a/Project.toml b/Project.toml index 362035eb7..5108d9aa9 100644 --- a/Project.toml +++ b/Project.toml @@ -1,6 +1,6 @@ name = "DynamicPPL" uuid = "366bfd00-2699-11ea-058f-f148b4cae6d8" -version = "0.36.11" +version = "0.36.12" [deps] ADTypes = "47edcb42-4c32-4615-8424-f2b9edc5f35b"