Conversation
…unctions in stocnet tutorials run without error or warning
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
Contributor
There was a problem hiding this comment.
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Testing
Tutorials
Checklist: