Skip to content

v1.6.2#314

Merged
jhollway merged 8 commits intomainfrom
develop
Apr 17, 2026
Merged

v1.6.2#314
jhollway merged 8 commits intomainfrom
develop

Conversation

@jhollway
Copy link
Copy Markdown
Collaborator

Description

Testing

  • Fixed namespace issues

Tutorials

  • Added function testing for migraph tutorials
  • Added function testing for manynet tutorials (> v2.0.2)
  • Updated examples in solutions in migraph tutorials where errors could creep in
  • Skipped extracting some solutions in migraph where errors could creep in

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 17, 2026 14:22
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.74%. Comparing base (2f70e63) to head (fe79f87).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
- Coverage   60.78%   60.74%   -0.05%     
==========================================
  Files          10        9       -1     
  Lines         709      698      -11     
==========================================
- Hits          431      424       -7     
+ Misses        278      274       -4     

☔ 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 bump to migraph v1.6.2, focusing on tutorial-related test improvements, shorter-running model tests, and a small namespace fix.

Changes:

  • Reworked tutorial tests by introducing new test helpers and adding “function execution” checks for migraph/manynet tutorials.
  • Updated several tutorial Rmds to reduce error-prone extraction/execution (e.g., purl=FALSE, adjusted examples, commented out problematic calls).
  • Reduced runtime of model prediction tests and fixed a namespace call in test_fit().

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
tests/testthat/test-tutorials_netrics.R Switch tutorial test helper to check_tute_rendering()
tests/testthat/test-tutorials_migraph.R Replace rendering test with code-execution checks via check_tute_functions()
tests/testthat/test-tutorials_manynet.R Add rendering + code-execution tutorial tests (gated by manynet version for execution)
tests/testthat/test-tutorials_autograph.R Switch tutorial test helper to check_tute_rendering()
tests/testthat/test-model_predict.R Reduce net_regression(..., times=) for faster tests
tests/testthat/test-model_distrib.R Update test_fit() test to reflect new coercion behavior
tests/testthat/helper-functions.R Add tutorial discovery + rendering + extracted-code execution helpers
man/test_tutorials.Rd Remove documentation for removed exported function
inst/tutorials/tutorial9/ergm.Rmd Mark slow/fragile chunk as purl=FALSE
inst/tutorials/tutorial8/diversity.Rmd Update solution example to use fict_marvel workflow
inst/tutorials/tutorial7/diffusion.Rmd Adjust examples and mark some chunks purl=FALSE / comment out problematic calls
inst/tutorials/tutorial0/tutorial0.Rmd Prevent extracting an intentionally failing example via purl=FALSE
cran-comments.md Update CRAN submission notes
R/tutorial_test.R Remove exported test_tutorials() implementation
R/model_distrib.R Fix namespace usage for is_graph() / as_diffusion()
NEWS.md Add 1.6.2 entry + adjust 1.6.1 date
NAMESPACE Stop exporting test_tutorials()
DESCRIPTION Bump version/date to 1.6.2 / 2026-04-17

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

Comment thread tests/testthat/helper-functions.R
Comment thread NAMESPACE
Comment thread NEWS.md
Comment thread tests/testthat/helper-functions.R Outdated
Comment thread tests/testthat/helper-functions.R
Comment thread tests/testthat/helper-functions.R
@jhollway jhollway merged commit c4919cc into main Apr 17, 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