Some simplifications

Author: penelopeysm

src/simple_varinfo.jl

@@ -234,9 +234,7 @@ end
 function SimpleVarInfo{T}(
     rng::Random.AbstractRNG, model::Model, init_strategy::AbstractInitStrategy=PriorInit()
 ) where {T<:Real}
-    new_context = setleafcontext(model.context, InitContext(rng, init_strategy))
-    new_model = contextualize(model, new_context)
-    return last(evaluate!!(new_model, SimpleVarInfo{T}()))
+    return last(init!!(rng, model, SimpleVarInfo{T}(), init_strategy))
 end
 function SimpleVarInfo{T}(
     model::Model, init_strategy::AbstractInitStrategy=PriorInit()
@@ -491,8 +489,9 @@ function settrans!!(vi::ThreadSafeVarInfo{<:SimpleVarInfo}, trans)
     return Accessors.@set vi.varinfo = settrans!!(vi.varinfo, trans)
 end
 function settrans!!(vi::SimpleOrThreadSafeSimple, trans::Bool, ::VarName)
-    # We keep this method around just to obey the AbstractVarInfo interface; however,
-    # this is only a valid operation if it would be a no-op.
+    # We keep this method around just to obey the AbstractVarInfo interface.
+    # However, note that this would only be a valid operation if it would be a
+    # no-op, which we check here.
     if trans != istrans(vi)
         error(
             "Individual variables in SimpleVarInfo cannot have different `settrans` statuses.",