Conversation
…d filter_ nodes and ties
…bs to startup messages
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #143 +/- ##
==========================================
- Coverage 42.70% 42.44% -0.27%
==========================================
Files 36 36
Lines 4334 4368 +34
==========================================
+ Hits 1851 1854 +3
- Misses 2483 2514 +31 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR bumps {manynet} to v2.1.2 and updates docs/startup messaging to reflect the migration of visualisation and analytic functionality into {autograph} and {netrics} (via {migraph}), while extending several manipulation/coercion functions with new S3 methods and improved “active” context handling (notably for stocnet).
Changes:
- Documentation/startup updates about migrated functionality (README, startup tips), plus minor reference/DOI formatting cleanups.
- New/updated S3 methods and dispatch paths (
filter_ties.igraph,filter_ties.stocnet,bind_changes.igraph,to_correlation.matrix) and new internal “active context” helpers used bystocnetnode/tie verbs. - Cleanup/removal of a set of defunct exports and corresponding Rd/NAMESPACE entries.
Reviewed changes
Copilot reviewed 13 out of 17 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| README.Rmd | Adds migration note and updates TOC to include new wrangling section. |
| README.md | Regenerates README with the migration note and TOC update. |
| R/zzz.R | Updates startup tips, including notes about {autograph}/{netrics} and {migraph}. |
| R/reexports_classes.R | Refactors expect_nodes()/expect_ties() and introduces internal active-context helpers. |
| R/modif_correlation.R | Adds to_correlation.matrix() S3 method. |
| R/manynet-defunct.R | Removes a set of defunct wrappers for moved/retired functions. |
| R/manip_ties.R | Adds filter_ties.igraph and filter_ties.stocnet; wraps mutate_ties.stocnet in active context. |
| R/manip_nodes.R | Wraps filter_nodes.stocnet and mutate_nodes.stocnet in active context. |
| R/manip_changes.R | Splits bind_changes logic into an igraph method and delegates tbl_graph to it. |
| R/make_generate.R | Reference formatting tweaks (DOI and citation text). |
| R/coerce_graph.R | Adjusts as_diffusion.igraph() to derive diffusion events from changelists. |
| NAMESPACE | Registers new S3 methods and removes exports for deleted defunct functions. |
| man/manip_ties_num.Rd | Updates method availability table for new filter_ties methods. |
| man/manip_changes.Rd | Updates method availability table for new bind_changes.igraph. |
| man/make_stochastic.Rd | Keeps Rd in sync with reference formatting/DOI changes. |
| man/defunct.Rd | Removes aliases/entries for removed defunct functions. |
| DESCRIPTION | Bumps version to 2.1.2 and updates title/description text. |
Files not reviewed (4)
- man/defunct.Rd: Generated file
- man/make_stochastic.Rd: Generated file
- man/manip_changes.Rd: Generated file
- man/manip_ties_num.Rd: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
Package
{autograph},{netrics}, and{migraph}Making
Coercion
expect_nodes()/expect_ties()and introduces internal active-context helpersas_diffusion.igraph()to derive diffusion events from changelistsManipulating
filter_ties.igraph()andfilter_ties.stocnet()mutate_nodes.stocnet(),mutate_ties.stocnet(),filter_nodes.stocnet(), andfilter_ties.stocnet()bind_changes.igraph()and delegated 'tbl_graph' method to itModifying
to_correlation.matrix()S3 methodChecklist: