Skip to content

Add vignettes, test functions, and indication-specific ORR-PFS updates - #1

Merged
yuliasidi merged 28 commits into
mainfrom
develop
Jun 9, 2026
Merged

Add vignettes, test functions, and indication-specific ORR-PFS updates#1
yuliasidi merged 28 commits into
mainfrom
develop

Conversation

@ShunRao8

Copy link
Copy Markdown
Collaborator
  1. Added vignettes covering:
  • PoS estimation example with indication-specific groups
  • Single-arm setting using lower/upper bounds on control ORR
  1. Added test functions for:
  • get_beta_params()
  • resp2oddsratio_estimate_ctrl()

@ShunRao8 ShunRao8 self-assigned this Mar 23, 2026
- Delete src/ directory (cleanup, cleanup.win, Makevars, install.libs.R,
  stan/ copies) -- these were generated by instantiate::stan_package_configure()
  and are no longer needed
- Remove stale src/stan/*.rds entry from .Rbuildignore
- Add R/globals.R with globalVariables() to suppress R CMD check NOTEs
  for NSE variables in gen_pos (omega, pos, reject, theta_P3_hat)
- Add R/stan_platform.R with two-tier Stan model cache:
  tier 1: inst/stan/precompiled/<platform_key>/ (bundled, read-only)
  tier 2: tools::R_user_dir('oncoPoS', 'cache')/<platform_key>/ (user-writable)
  Fixes recompilation every session on Linux system library installs
- Fix non-ASCII em/en dash characters in roxygen comments across gen_pos.R,
  run_stan.R, and stan_platform.R (caused devtools::document() UTF-8 error)

@yuliasidi yuliasidi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ShunRao8 , were you able to reproduce the mean(posterior_draws$sigma_wls) number based on the published data?

If, yes, I will approve the PR. You can see in the commit history some minor changes + updates for installing this more smoothly on Windows

@ShunRao8

ShunRao8 commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator Author

@yuliasidi Thank you! Yes, I was able to reproduce the value. Ready for your approval.

@yuliasidi
yuliasidi merged commit d460d4c into main Jun 9, 2026
6 checks passed
@yuliasidi
yuliasidi deleted the develop branch June 9, 2026 13:08
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