Skip to content

v1.6.1#313

Merged
jhollway merged 12 commits intomainfrom
develop
Apr 14, 2026
Merged

v1.6.1#313
jhollway merged 12 commits intomainfrom
develop

Conversation

@jhollway
Copy link
Copy Markdown
Collaborator

Description

Package

  • Fixed logo sizing and location

Tutorials

  • Fixed run_tute() to identify and use tutorials in {netrics}
  • Updated diversity tutorial with new netrics names
  • Updated diffusion tutorial with new netrics names
  • Updated ergm tutorial with new netrics names

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"

Copilot AI review requested due to automatic review settings April 13, 2026 13:25
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 13, 2026

Codecov Report

❌ Patch coverage is 63.15789% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.78%. Comparing base (129a8f3) to head (2f70e63).
⚠️ Report is 22 commits behind head on main.

Files with missing lines Patch % Lines
R/tutorial_run.R 50.00% 6 Missing ⚠️
R/model_distrib.R 85.71% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #313      +/-   ##
==========================================
+ Coverage   52.58%   60.78%   +8.20%     
==========================================
  Files          10       10              
  Lines         696      709      +13     
==========================================
+ Hits          366      431      +65     
+ Misses        330      278      -52     

☔ 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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Release prep for migraph 1.6.1, focusing on tutorial compatibility with {netrics} and updating tutorial content/function naming, along with the corresponding version bump and changelog entry.

Changes:

  • Add {netrics} to the tutorial discovery/run logic (run_tute()).
  • Update tutorial Rmd/HTML content to use the newer net_by_* / *_by_* function names.
  • Bump package version/date and add a 1.6.1 entry to NEWS.md.

Reviewed changes

Copilot reviewed 5 out of 11 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
R/tutorial_run.R Adds netrics to tutorial package list and run order (but introduces an extract_tute() gap for netrics).
NEWS.md Adds release notes for 1.6.1.
inst/tutorials/tutorial8/diversity.Rmd Updates tutorial code to new netrics function naming.
inst/tutorials/tutorial8/diversity.html Re-rendered tutorial output reflecting code changes, but contains stale prose references.
inst/tutorials/tutorial7/diffusion.Rmd Updates tutorial code to new netrics function naming, but leaves stale prose references.
DESCRIPTION Version/date bumped to 1.6.1 / 2026-04-13.

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

Comment thread R/tutorial_run.R
Comment thread R/tutorial_run.R
Comment on lines 39 to +42
try(learnr::run_tutorial(tute, "manynet"), silent = TRUE)
try(learnr::run_tutorial(tute, "migraph"), silent = TRUE)
try(learnr::run_tutorial(tute, "autograph"), silent = TRUE)
try(learnr::run_tutorial(tute, "netrics"), silent = TRUE)
try(learnr::run_tutorial(tute, "migraph"), silent = TRUE)
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

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

run_tute() now adds an additional package lookup (netrics), but there are no automated tests covering the tutorial resolution / package iteration logic. Consider adding a small test that mocks learnr::run_tutorial (e.g., via testthat::local_mocked_bindings) to assert that run_tute() attempts the expected packages (including netrics) without actually launching a tutorial.

Copilot uses AI. Check for mistakes.
Comment thread inst/tutorials/tutorial7/diffusion.Rmd Outdated
Comment thread inst/tutorials/tutorial7/diffusion.Rmd
Comment thread inst/tutorials/tutorial7/diffusion.Rmd
Comment thread inst/tutorials/tutorial8/diversity.Rmd
Comment thread inst/tutorials/tutorial8/diversity.html
@jhollway jhollway merged commit b42a238 into main Apr 14, 2026
5 of 6 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