Closed krivit closed 1 year ago
Also, in the controls set by ergm.stocapprox()
, use more descriptive names. For example, there is no reason to replace SA.nsubphases
with nsub
as it currently does.
In the stoch.approx(), Is the argument "s.obs" related to the missing data? Why does it use !is.null(s.obs) ?
s.obs
is an ergm_state
object that is only non-NULL
if there is a missing data process. It's passed because missing data support for SA is on the roadmap.
Currently, many of the
control.ergm()
'sSA.*
arguments are set toNULL
and substituted inergm.stocapprox()
.Instead, they should be set to the defaults there. Where they are specific to the model or similar, they should be functions along the lines of
init.MPLE.samplesize=
.