Remove @submodel

Author: penelopeysm

HISTORY.md

@@ -4,6 +4,10 @@
 
 **Breaking changes**
 
+### Submodel macro
+
+The `@submodel` macro is fully removed; please use `to_submodel` instead.
+
 ### Accumulators
 
 This release overhauls how VarInfo objects track variables such as the log joint probability. The new approach is to use what we call accumulators: Objects that the VarInfo carries on it that may change their state at each `tilde_assume!!` and `tilde_observe!!` call based on the value of the variable in question. They replace both variables that were previously hard-coded in the `VarInfo` object (`logp` and `num_produce`) and some contexts. This brings with it a number of breaking changes:

docs/src/api.md

@@ -146,12 +146,6 @@ to_submodel
 
 Note that a `[to_submodel](@ref)` is only sampleable; one cannot compute `logpdf` for its realizations.
 
-In the past, one would instead embed sub-models using [`@submodel`](@ref), which has been deprecated since the introduction of [`to_submodel(model)`](@ref)
-
-```@docs
-@submodel
-```
-
 In the context of including models within models, it's also useful to prefix the variables in sub-models to avoid variable names clashing:
 
 ```@docs

src/DynamicPPL.jl

@@ -128,7 +128,6 @@ export AbstractVarInfo,
     to_submodel,
     # Convenience macros
     @addlogprob!,
-    @submodel,
     value_iterator_from_chain,
     check_model,
     check_model_and_trace,
@@ -187,7 +186,6 @@ include("simple_varinfo.jl")
 include("context_implementations.jl")
 include("compiler.jl")
 include("pointwise_logdensities.jl")
-include("submodel_macro.jl")
 include("transforming.jl")
 include("logdensityfunction.jl")
 include("model_utils.jl")

src/submodel_macro.jl

@@ -72,7 +72,6 @@ include("test_util.jl")
         include("context_implementations.jl")
         include("threadsafe.jl")
         include("debug_utils.jl")
-        include("deprecated.jl")
         include("submodels.jl")
         include("bijector.jl")
     end