Skip to content

v1.0.3#55

Merged
jhollway merged 2 commits into
mainfrom
develop
May 1, 2026
Merged

v1.0.3#55
jhollway merged 2 commits into
mainfrom
develop

Conversation

@jhollway

@jhollway jhollway commented May 1, 2026

Copy link
Copy Markdown
Contributor

…resby serializing it, moving to extdata, and loading it viaload_ergm_res()`

Description

Checklist:

  • Documentation
    • DESCRIPTION file version is bumped by the appropriate increment (major, minor, patch)
    • Date in DESCRIPTION is correct
    • Longer functions are commented inline or broken down into helper functions to help debugging
  • PR form
    • Title indicates expected version number
    • PR description above and the NEWS.md file are aligned
    • Description above itemizes changes under subsection titles, e.g. "## Data""
    • Closed, fixed, or related issues are referenced and explained in the description above, e.g. "Fixed #0 by adding A"

…res` by serializing it, moving to extdata, and loading it via `load_ergm_res()`
Copilot AI review requested due to automatic review settings May 1, 2026 11:00
@codecov

codecov Bot commented May 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.32%. Comparing base (eca96cd) to head (3838387).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
+ Coverage   56.30%   56.32%   +0.02%     
==========================================
  Files          24       24              
  Lines        1920     1921       +1     
==========================================
+ Hits         1081     1082       +1     
  Misses        839      839              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the package’s bundled ERGM example result object to avoid a namespace-reference error by moving it out of data/ and into inst/extdata/, and adds a small helper to load it for examples/tests.

Changes:

  • Added load_ergm_res() (exported) to load the serialized ERGM result from inst/extdata/ergm_res.rds.
  • Updated tests and examples to use load_ergm_res() instead of relying on a data() dataset.
  • Removed ergm_res from the “precooked datasets” documentation and bumped package version/date for 1.0.3.

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
R/plot_convergence.R Adds load_ergm_res() and updates examples for ERGM convergence plotting.
inst/extdata/ergm_res.rds Adds the serialized ERGM result used by load_ergm_res().
tests/testthat/test-plot_convergence.R Updates the ERGM diagnostics test to load the ERGM result via load_ergm_res().
R/data_precooked.R Removes roxygen dataset entry for ergm_res.
man/made_earlier.Rd Drops ergm_res from the precooked datasets help page.
man/plot_convergence.Rd Documents load_ergm_res() on the convergence plotting help page and updates examples.
NAMESPACE Exports load_ergm_res.
NEWS.md Adds 1.0.3 changelog entry describing this fix.
DESCRIPTION Bumps version to 1.0.3 and updates the date.
data/ergm_res.rda Removed (no longer shipped as a data() dataset).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread R/plot_convergence.R
#' @rdname plot_convergence
#' @export
load_ergm_res <- function() {
readRDS(file = system.file("extdata", "ergm_res.rds", package = "autograph"))
Comment thread R/plot_convergence.R
Comment on lines +85 to +89
#' @rdname plot_convergence
#' @export
load_ergm_res <- function() {
readRDS(file = system.file("extdata", "ergm_res.rds", package = "autograph"))
}
Comment thread NEWS.md

## Plotting

- Fixed the error produced by a namespace reference in a call in `ergm_res` by serializing it, moving to extdata, and loading it via `load_ergm_res()`
@jhollway jhollway merged commit b71432f into main May 1, 2026
3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants