InitContext isn't for 0.37, update comments

Author: penelopeysm

src/mcmc/abstractmcmc.jl

@@ -1,7 +1,7 @@
 # TODO: Implement additional checks for certain samplers, e.g.
 # HMC not supporting discrete parameters.
 function _check_model(model::DynamicPPL.Model)
-    # TODO(DPPL0.37/penelopeysm): use InitContext
+    # TODO(DPPL0.38/penelopeysm): use InitContext
     spl_model = DynamicPPL.contextualize(model, DynamicPPL.SamplingContext(model.context))
     return DynamicPPL.check_model(spl_model, VarInfo(); error_on_failure=true)
 end

src/mcmc/emcee.jl

@@ -53,7 +53,7 @@ function AbstractMCMC.step(
         length(initial_params) == n ||
             throw(ArgumentError("initial parameters have to be specified for each walker"))
         vis = map(vis, initial_params) do vi, init
-            # TODO(DPPL0.37/penelopeysm) This whole thing can be replaced with init!!
+            # TODO(DPPL0.38/penelopeysm) This whole thing can be replaced with init!!
             vi = DynamicPPL.initialize_parameters!!(vi, init, model)
 
             # Update log joint probability.

src/mcmc/ess.jl

@@ -84,7 +84,7 @@ EllipticalSliceSampling.isgaussian(::Type{<:ESSPrior}) = true
 function Base.rand(rng::Random.AbstractRNG, p::ESSPrior)
     varinfo = p.varinfo
     # TODO: Surely there's a better way of doing this now that we have `SamplingContext`?
-    # TODO(DPPL0.37/penelopeysm): This can be replaced with `init!!(p.model,
+    # TODO(DPPL0.38/penelopeysm): This can be replaced with `init!!(p.model,
     # p.varinfo, PriorInit())` after TuringLang/DynamicPPL.jl#984. The reason
     # why we had to use the 'del' flag before this was because
     # SampleFromPrior() wouldn't overwrite existing variables.

src/mcmc/mh.jl

@@ -178,7 +178,7 @@ get_varinfo(s::MHState) = s.varinfo
 # Utility functions #
 #####################
 
-# TODO(DPPL0.37/penelopeysm): This function should no longer be needed
+# TODO(DPPL0.38/penelopeysm): This function should no longer be needed
 # once InitContext is merged.
 """
     set_namedtuple!(vi::VarInfo, nt::NamedTuple)
@@ -207,7 +207,7 @@ end
 # NOTE(penelopeysm): MH does not conform to the usual LogDensityProblems
 # interface in that it gets evaluated with a NamedTuple. Hence we need this
 # method just to deal with MH.
-# TODO(DPPL0.37/penelopeysm): Check the extent to which this method is actually
+# TODO(DPPL0.38/penelopeysm): Check the extent to which this method is actually
 # needed. If it's still needed, replace this with `init!!(f.model, f.varinfo,
 # ParamsInit(x))`. Much less hacky than `set_namedtuple!` (hopefully...).
 # In general, we should much prefer to either (1) conform to the

src/mcmc/prior.jl

@@ -12,7 +12,7 @@ function AbstractMCMC.step(
     state=nothing;
     kwargs...,
 )
-    # TODO(DPPL0.37/penelopeysm): replace with init!!
+    # TODO(DPPL0.38/penelopeysm): replace with init!!
     sampling_model = DynamicPPL.contextualize(
         model, DynamicPPL.SamplingContext(rng, DynamicPPL.SampleFromPrior(), model.context)
     )