-
Notifications
You must be signed in to change notification settings - Fork 22
Add degree and time of adoption diagnostic function #57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
3b20049
Initial plan
Copilot a6c7fed
Implement degree_adoption_diagnostic function with comprehensive tests
Copilot 42b6876
Fix print method formatting and complete implementation
Copilot c12576e
Review changes:
aoliveram 32de3fd
multi-diffusion support (behavior/combine), undirected network suppor…
aoliveram 48be376
Refactor degree_adoption_diagnostic: Move to separate file, update do…
Copilot 72234a2
Updating docs
gvegayon d193bec
Adding new workflow and working on R CMD check
gvegayon 30818c0
Fixing actions
gvegayon 5a90114
Updating readme
gvegayon f7eb8fe
Fix formatting of valgrind variable in workflow name
gvegayon 5c83647
Addressing errors and warnings
aoliveram e2c03df
Updating printer: combo mode + bootstrap NA fix
aoliveram d5d850b
Merge branch 'master' into copilot/fix-56
gvegayon 11102b5
Updating README and version number
gvegayon 59d54b3
Remove build-targz.yml and update NEWS.md for v1.24.0
Copilot fc7cecb
Renaming codechunks to avoid errors
gvegayon 4a3711f
Merge branch 'copilot/fix-56' of https://github.com/USCCANA/netdiffus…
gvegayon 2e9de66
Trying to fix LR ending
gvegayon ff009bc
Avoiding building manual in valgrind and building to upload
gvegayon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,16 +1,16 @@ | ||
| FROM rocker/tidyverse:4.4.0 | ||
| FROM ghcr.io/rocker-org/devcontainer/tidyverse:4.5 | ||
|
|
||
| # RUN \ | ||
| # echo 'options(repos=c(CRAN="https://packagemanager.posit.co/cran/__linux__/bookworm/latest"))' >> ~/.Rprofile && \ | ||
| # Rscript --vanilla -e 'getOption("repos")' | ||
|
|
||
| # Adding R packages | ||
| RUN install2.r Rcpp sna network networkDynamic Matrix MASS MatchIt SparseM igraph \ | ||
| viridisLite covr testthat knitr rmarkdown ape RSiena survival RcppArmadillo | ||
|
|
||
| RUN install2.r languageserver httpgd | ||
| RUN install2.r --error languageserver && installGithub.r nx10/httpgd | ||
|
|
||
| RUN apt-get update && apt-get install --no-install-recommends -y valgrind gdb \ | ||
| libglpk-dev | ||
|
|
||
| # Adding R packages | ||
| RUN install2.r --error Rcpp sna network networkDynamic Matrix MASS MatchIt SparseM igraph \ | ||
| viridisLite covr testthat knitr rmarkdown ape RSiena survival RcppArmadillo | ||
|
|
||
| CMD ["bash"] |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| # Copilot Instructions for netdiffuseR | ||
|
|
||
| ## Documentation Style | ||
|
|
||
| When writing roxygen2 documentation, prefer markdown syntax over LaTeX: | ||
|
|
||
| - Use `` `code` `` instead of `\code{code}` | ||
| - Use `` `[function()]` `` instead of `\code{\link{function}}` | ||
| - Use markdown bullet lists instead of `\itemize{}` | ||
| - Use markdown formatting for emphasis and structure | ||
|
|
||
| ## Code Style | ||
|
|
||
| - Follow existing code patterns and conventions in the package | ||
| - Use meaningful variable names and keep functions focused | ||
| - Add comments only when they match the existing style or explain complex logic | ||
| - Prefer using existing libraries over adding new dependencies | ||
|
|
||
| ## Testing | ||
|
|
||
| - Add comprehensive tests for new functions | ||
| - Include edge cases and error conditions | ||
| - Test with the existing sample datasets when possible | ||
| - Validate input parameters and handle errors gracefully | ||
|
|
||
| ## Examples | ||
|
|
||
| - Use `\dontrun{}` only when examples take a long time to run | ||
| - Prefer examples that can execute quickly for CRAN checks | ||
| - Use existing package datasets in examples when available |
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -22,3 +22,8 @@ inst/doc | |
| *.tex | ||
| playground/ | ||
| doc/ | ||
|
|
||
| # Other files | ||
| .vscode/ | ||
| config.status | ||
| src/Makevars | ||
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to fill in the blanks. Ensure to add what's new from this version vs the previous one. You can use GitHub to compare differences and perhaps update the NEWS. LMK if you need help with that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated NEWS.md to document the new
degree_adoption_diagnostic()function in version 1.24.0 (DEV). The entry was moved from v1.23.0 and expanded to include details about multi-diffusion support, undirected networks, and bootstrap confidence intervals. See commit 59d54b3.