Skip to content

v1.7.0 - #322

Merged
jhollway merged 13 commits into
mainfrom
develop
Aug 30, 2026
Merged

v1.7.0#322
jhollway merged 13 commits into
mainfrom
develop

Conversation

@jhollway

Copy link
Copy Markdown
Member

Package

  • Raised the minimum versions of the stocnet dependencies: manynet 2.3.1, autograph 1.2.2, and netrics 1.0.1
  • Updated CONTRIBUTING

Models

Data

  • Converted every mpn_* dataset to the stocnet class, in line with {manynet}
  • Removed the weight tie attribute from the datasets that are not weighted
  • Merged mpn_cow
  • Merged mpn_senate
    • This combines the 112th Congress data into one multilevel network
      of 93 senators, 78 PACs, and 25 bills
    • mpn_senate_dem and mpn_senate_rep are not party networks but
      record two communities of one network and their intersection;
      the community node attribute holds "A", "B", or "both"
    • Cells that neither community covers were never observed, and are recorded as
      missing rather than as absent ties
    • Where the two communities disagree, in 9 of their 640 shared cells,
      community A governs, since it agrees with the overlap file in all 640
    • mpn_senate_dem, mpn_senate_rep, and mpn_senate_over are deprecated and
      will be removed in 1.8.0
  • Merged mpn_evs_ita, mpn_evs_deu, and mpn_evs_gbr
    • These combine each country's European Values Study data for 1990 and 2008 into one two-mode network,
      with the year recorded in the time tie attribute
    • These are repeated cross-sections, not a panel, so use over_time() not over_waves()
    • The six year-by-country datasets are deprecated and will be removed in 1.8.0
  • Converted mpn_elite_mex
  • Converted mpn_elite_usa_advice
    • It now names its two modes, "directors" and "think tanks"
  • Converted mpn_elite_usa_money
    • It now names its three modes, "elites", "organisations", and "candidates"
  • Converted mpn_bristol
    • It now names its three modes, "individuals", "organisations", and "events"
  • Converted mpn_ryanair

Copilot AI lite review requested due to automatic review settings August 30, 2026 05:48
@jhollway jhollway self-assigned this Aug 30, 2026

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 prepares the migraph v1.7.0 release by bumping minimum dependency versions, improving diffusion simulation compatibility with stocnet inputs, and updating/expanding dataset documentation (including new combined multilevel/longitudinal mpn_* datasets and deprecation notices).

Changes:

  • Bumped package version to 1.7.0 and raised minimum {manynet}, {autograph}, and {netrics} versions.
  • Fixed play_diffusions() for stocnet inputs by coercing to igraph prior to calling {manynet} diffusion logic.
  • Updated dataset docs/NEWS/CONTRIBUTING and improved the release workflow to populate GitHub Release notes from NEWS.md.

Reviewed changes

Copilot reviewed 16 out of 39 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
R/model_diffusion.R Adds stocnet coercion for play_diffusions() compatibility.
R/data_mpn.R Adds/updates roxygen docs for merged/converted mpn_* datasets and deprecations.
NEWS.md Adds release notes section for 1.7.0.
DESCRIPTION Bumps version to 1.7.0 and raises minimum dependency versions.
cran-comments.md Updates CRAN submission notes (incl. reverse-dep context and formatting).
.github/workflows/pushrelease.yml Extracts release notes from NEWS.md and uses them as the GitHub Release body.
.github/CONTRIBUTING.md Expands contributing guidance, including dataset conventions and workflow expectations.
man/make_play.Rd Documents stocnet as an accepted input type.
man/mpn_senate.Rd New dataset documentation for merged mpn_senate.
man/mpn_senate112.Rd Updates docs for superseded mpn_senate_* community datasets.
man/mpn_evs_combined.Rd New docs for combined EVS longitudinal datasets.
man/mpn_evs.Rd Updates docs for superseded EVS year-by-country datasets.
man/mpn_cow_combined.Rd New docs for merged mpn_cow.
man/mpn_cow_separate.Rd Renames/separates docs for mpn_cow_trade/mpn_cow_igo alongside merged dataset.
man/mpn_elite_usa.Rd Updates docs reflecting mode naming/layering for US elite datasets.
man/mpn_elite_mex.Rd Updates docs reflecting stocnet-style node naming.
man/mpn_bristol.Rd Updates docs reflecting stocnet conversion and mode naming.
man/mpn_ryanair.Rd Updates docs reflecting stocnet conversion (node/tie field changes).
Files not reviewed (2)
  • man/make_play.Rd: Generated file
  • man/mpn_bristol.Rd: Generated file
Suppressed comments (2)

R/model_diffusion.R:42

  • furrr::furrr_options(seed = T) uses the shorthand constant T, which can be overridden by user code (it’s just a variable) and is less robust than TRUE. Use seed = TRUE to avoid accidental non-reproducibility.
  if (inherits(.data, "stocnet")) .data <- manynet::as_igraph(.data)
  
  out <- furrr::future_map_dfr(1:times, function(j){
    data.frame(sim = j,
               manynet::as_diffusion(manynet::play_diffusion(.data, ...)))
  }, .progress = verbose, .options = furrr::furrr_options(seed = T))

R/data_mpn.R:527

  • Typo in author name in the @references list below: “Nordstron” should be “Nordstrom” (Pevehouse et al. 2020).
#' @rdname mpn_cow_separate
#' @usage data(mpn_cow_igo)
#' @source
#' The Correlates of War Project. 2019. \emph{Intergovernmental Organization v3}.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread R/data_mpn.R Outdated
Comment thread man/mpn_senate.Rd
Comment on lines +8 to +10
\if{html}{\out{<div class="sourceCode">}}\preformatted{#> -- # Senate contribution and voting network ------------------------------------
#> # A labelled, undirected network of 196 senators and 5861 contribution and vote
#> ties
Comment thread man/mpn_bristol.Rd
Comment on lines 8 to +10
\if{html}{\out{<div class="sourceCode">}}\preformatted{#> -- # Bristol protest event network ---------------------------------------------
#> # A labelled, two-mode network of 150 individuals and 114 individuals and 1496
#> affiliation and participation ties
#> # A labelled, undirected network of 264 individuals and 1496 affiliation and
#> participation ties
Comment thread man/mpn_elite_usa.Rd
Comment on lines +40 to 43
\if{html}{\out{<div class="sourceCode">}}\preformatted{#> -- # US power elite network ----------------------------------------------------
#> # A labelled, multiplex, undirected network of 38 elites and 59 board
#> membership ties and 44 campaign contribution ties
#>
Comment thread man/mpn_senate112.Rd
Comment on lines +11 to +13
\if{html}{\out{<div class="sourceCode">}}\preformatted{#> -- # Senate community A --------------------------------------------------------
#> # A labelled, undirected network of 114 senators and 2791 contribution and vote
#> ties
@codecov

codecov Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.74%. Comparing base (cd91db3) to head (8d460da).
⚠️ Report is 18 commits behind head on main.

Files with missing lines Patch % Lines
R/model_diffusion.R 0.00% 1 Missing ⚠️
R/zzz.R 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #322      +/-   ##
==========================================
+ Coverage   61.17%   61.74%   +0.57%     
==========================================
  Files           9       10       +1     
  Lines         716      732      +16     
==========================================
+ Hits          438      452      +14     
- Misses        278      280       +2     

☔ View full report in Codecov by Harness.
📢 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.

@jhollway
jhollway merged commit 7cef4be into main Aug 30, 2026
7 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