Skip to content

Rename amRshiny references to amRviz#3

Merged
eboyer221 merged 1 commit into
mainfrom
dev-rename-amRshiny-to-amRviz
May 27, 2026
Merged

Rename amRshiny references to amRviz#3
eboyer221 merged 1 commit into
mainfrom
dev-rename-amRshiny-to-amRviz

Conversation

@eboyer221
Copy link
Copy Markdown
Collaborator

#Description
The amRshiny package has been renamed to amRviz. This PR updates all references to the old name across the amR package suite.

##Changes

  • DESCRIPTION - updated title and description text
  • R/install_amR.R - updated roxygen docs; renamed the packages argument value "shiny" → "viz"
  • updated valid_packages and repo_map to point at JRaviLab/amRviz
  • man/installAMR.Rd - regenerated to match
  • README.Rmd / README.md - updated install commands, quick-start, workflow diagram, and documentation links
  • vignettes/intro.Rmd - updated package list and workflow steps
  • tests/testthat/test-install.R - updated expected defaults and valid-package vectors

##Note to Users
installAMR(packages = "shiny") no longer works; use installAMR(packages = "viz"). Users who call installAMR() with no arguments are unaffected, since the default already includes "viz".

##To Test

  • devtools::check() passes (0 errors / 0 warnings / 0 notes)
  • devtools::test() passes
  • installAMR() with defaults pulls JRaviLab/amRviz (not amRshiny)
  • installAMR(packages = "viz") works; installAMR(packages = "shiny") errors as expected
  • README on GitHub shows amRviz everywhere

Copy link
Copy Markdown
Member

@jananiravi jananiravi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Emily, please make sure the replacement image is at least 600 dpi. The rest of it looks good -- seems straightforward!

@eboyer221 eboyer221 merged commit 1366c13 into main May 27, 2026
7 of 8 checks passed
@eboyer221 eboyer221 deleted the dev-rename-amRshiny-to-amRviz branch May 27, 2026 17:12
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