Conversation
…aw fit beyond igraph algorithm
…e clearer about inheritance and avoid base function conflicts
…ime() and node_by_adopt_threshold() respectively
|
@copilot can you please fill in NEWS.md outlining the changes in this PR? |
Co-authored-by: jhollway <5595229+jhollway@users.noreply.github.com> Agent-Logs-Url: https://github.com/stocnet/netrics/sessions/884854c5-d15b-40c2-9e84-62c1fbba2040
There was a problem hiding this comment.
Pull request overview
This PR prepares the netrics v0.2.0 release by reorganizing documentation (roxygen templates + generated Rd), refining function grouping/naming (notably change metrics moving from net_by_* measures to net_x_* motifs), and updating pkgdown reference structure and tests accordingly.
Changes:
- Bumps package version/date to 0.2.0 (2026-03-21) and updates CRAN submission notes.
- Refactors/documentation reorganization: adds
man-roxygen/templates, splits/renames doc topics (e.g., brokerage motifs into node vs network topics), and updates pkgdown reference indexing. - Adjusts tests and README function listings to match renamed/recategorized APIs (e.g., diffusion “adopt” naming, change motifs).
Reviewed changes
Copilot reviewed 115 out of 115 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| DESCRIPTION | Version/date bumped for v0.2.0 release. |
| R/measure_change.R | Moves change-related APIs to net_x_* motif-style outputs and updates roxygen grouping. |
| R/measure_features.R | Updates net_by_scalefree() messaging/KS handling (contains a critical issue). |
| R/member_core.R | Splits docs into mark/measure/member sections and renames node_is_core() arg to centrality. |
| R/motif_brokerage.R | Splits brokerage motifs docs into node vs network topics; expands templates usage. |
| man-roxygen/* | Introduces reusable roxygen templates for params/returns (some templates/docs currently inconsistent). |
| pkgdown/_pkgdown.yml | Reorganizes reference index entries to match new topic structure. |
| tests/testthat/* | Updates test routing for renamed/regrouped functions (motif vs measure). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5 +/- ##
==========================================
- Coverage 81.94% 81.07% -0.88%
==========================================
Files 21 24 +3
Lines 2360 2394 +34
==========================================
+ Hits 1934 1941 +7
- Misses 426 453 +27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: jhollway <5595229+jhollway@users.noreply.github.com> Agent-Logs-Url: https://github.com/stocnet/netrics/sessions/cd56c0c0-bec1-4ac8-97a7-c63ab078b73c
Description
Package
param_attr,param_data,param_dir,param_memb,param_motf,param_norm,param_select) andnet/node/tie-level templates (
net_measure,net_motif,node_mark,node_measure,node_member,node_motif,tie_mark,tie_measure) forconsistent function documentation.
Measuring
node_adoption_time()tonode_by_adopt_time()node_thresholds()tonode_by_adopt_threshold()(betweenness, closeness, degree, eigenvector) for easier maintenance,
and into different documentation per level (node, tie) for better organization.
cliques, components, features, and hierarchy documentation by level.
Members
node_in_community()documentation from the hierarchicaland non-hierarchical community-detection algorithms.
Motifs
net_by_change()tonet_x_change()and related functions to reflect their motif (subgraph-counting) nature.Methods
method_k().model_k()tomethod_k()and related cluster-selection utilitiesrenamed for clarity.
Checklist: