Conversation
Codecov Report❌ Patch coverage is
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. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
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.
| 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) |
There was a problem hiding this comment.
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.
Description
Package
Tutorials
run_tute()to identify and use tutorials in{netrics}Checklist: