Revert "Implement returned(model, xs...)"

Accidentally committed to main, meant to open a PR.

This reverts commit 7bdc71e.

Author: penelopeysm

HISTORY.md

@@ -1,9 +1,5 @@
 # DynamicPPL Changelog
 
-## 0.39.2
-
-`returned(model, parameters...)` now accepts any arguments that can be wrapped in `InitFromParams` (previously it would only accept `NamedTuple`, `AbstractDict{<:VarName}`, or a chain).
-
 ## 0.39.1
 
 `LogDensityFunction` now allows you to call `logdensity_and_gradient(ldf, x)` with `AbstractVector`s `x` that are not plain Vectors (they will be converted internally before calculating the gradient).

Project.toml

@@ -1,6 +1,6 @@
 name = "DynamicPPL"
 uuid = "366bfd00-2699-11ea-058f-f148b4cae6d8"
-version = "0.39.2"
+version = "0.39.1"
 
 [deps]
 ADTypes = "47edcb42-4c32-4615-8424-f2b9edc5f35b"

src/model.jl

@@ -1153,16 +1153,10 @@ end
 function predict end
 
 """
-    returned(model::Model, parameters...)
+    returned(model::Model, parameters::NamedTuple)
+    returned(model::Model, parameters::AbstractDict{<:VarName})
 
-Initialise a `model` using the given `parameters` and return the model's return value. The
-parameters must be provided in a format that can be wrapped in an `InitFromParams`, i.e.,
-`InitFromParams(parameters..., nothing)` must be a valid `AbstractInitStrategy` (where
-`nothing` is the fallback strategy to use if parameters are not provided).
-
-As far as DynamicPPL is concerned, `parameters` can be either a singular `NamedTuple` or an
-`AbstractDict{<:VarName}`; however this method is left flexible to allow for other packages
-that wish to extend `InitFromParams`.
+Execute `model` with variables `keys` set to `values` and return the values returned by the `model`.
 
 # Example
 ```jldoctest
@@ -1183,7 +1177,7 @@ julia> returned(model, Dict{VarName,Float64}(@varname(m) => 2.0))
 (mp1 = 3.0,)
 ```
 """
-function returned(model::Model, parameters::Any)
+function returned(model::Model, parameters::Union{NamedTuple,AbstractDict{<:VarName}})
     # Note: we can't use `fix(model, parameters)` because
     # https://github.com/TuringLang/DynamicPPL.jl/issues/1097
     return first(
@@ -1192,7 +1186,7 @@ function returned(model::Model, parameters::Any)
             DynamicPPL.OnlyAccsVarInfo(DynamicPPL.AccumulatorTuple()),
             # Use `nothing` as the fallback to ensure that any missing parameters cause an
             # error
-            InitFromParams(parameters..., nothing),
+            InitFromParams(parameters, nothing),
         ),
     )
 end