Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

btergm's usage of ergm::ergm.pl() uses an outdated API #35

Open
krivit opened this issue May 27, 2023 · 1 comment
Open

btergm's usage of ergm::ergm.pl() uses an outdated API #35

krivit opened this issue May 27, 2023 · 1 comment

Comments

@krivit
Copy link

krivit commented May 27, 2023

I've added compatibility code for the upcoming release, but please switch to passing ergm_state objects.

@krivit
Copy link
Author

krivit commented Jan 20, 2025

Please address this ASAP. Since it's been almost 2 years, I am removing the compatibility code in an upcoming release.

By the way, it looks like you are suppressing deprecation warnings here:

test_that("offset argument in btergm works without composition change", {
skip_if_not_installed("fastglm", minimum_version = "0.0.1")
set.seed(12345)
fit1 <- suppressWarnings(btergm(networks ~ edges + istar(2) + edgecov(covariates),
R = 100,
offset = FALSE,
usefastglm = TRUE,
verbose = FALSE))
set.seed(12345)
fit2 <- suppressWarnings(btergm(networks ~ edges + istar(2) + edgecov(covariates),
R = 100,
offset = TRUE,
usefastglm = TRUE,
verbose = FALSE))
expect_equal(confint(fit1), confint(fit2))
})

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant