-
Notifications
You must be signed in to change notification settings - Fork 0
Add prior predictive check utilities for Bayesian workflow validation #178
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
base: main
Are you sure you want to change the base?
Conversation
…unctions with tests Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
|
Version update (from version-check failure) : Please bump the package development version (e.g., from 0.0.0.9047 to 0.0.0.9048) using WORDLIST update (from spellcheck failure) : Please add the following domain-specific terms to pkgdown build issue (gh-pages / reference index failure) : The pkgdown build fails because several exported functions and one article are missing from Lint fixes (from lint-changed-files) : Please fix lint warnings by correcting indentation, wrapping lines to ≤80 characters, replacing NEWS.md update (from changelog check failure) : Please add a NEWS.md entry describing the user-visible changes introduced in this PR, or apply the @copilot figure out the issues above |
…te pkgdown config, and add NEWS entry Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
All issues addressed in commit 1410f0b: ✅ Version update: Bumped to 0.0.0.9048 using ✅ WORDLIST update: Added MFI, PPCs, Shigella, and titers to ✅ pkgdown build: Added all three new functions ( ✅ Lint fixes:
✅ NEWS.md entry: Added comprehensive description of new PPC utilities under development version All tests passing (71 tests), no spelling errors. |
|
@copilot Please resolve remaining lint warnings: fix indentation in |
…r to params, add @Keywords internal to print method Co-authored-by: Kwan-Jenny <68584166+Kwan-Jenny@users.noreply.github.com>
All issues resolved in commit 733f223: ✅ Indentation fixes:
✅ Line wrapping: Lines 111-112 in test-summarize_prior_predictive.R wrapped to ≤80 chars ✅ Refactored par() issue: Renamed variable ✅ Replaced sapply(): Line 37 in test-simulate_prior_predictive.R now uses vapply() with FUN.VALUE = logical(1) ✅ Line length fixes: R/simulate_prior_predictive.R lines 168, 171 - moved inline comments to separate lines ✅ pkgdown build:
All linting warnings resolved (0 warnings in changed files), all 71 tests passing. |
|
📖 https://ucd-serg.github.io/serodynamics/preview/pr178 |
Prior Predictive Check (PPC) Implementation - COMPLETE
All PR Feedback Addressed ✓
Latest fixes (commit)
partoparamsin R/simulate_prior_predictive.R to avoid linter confusion with graphics::par()Previous fixes
Core Implementation - Completed ✓
simulate_prior_predictive()- Simulates trajectories from priorssummarize_prior_predictive()- Diagnostic summaries with print methodplot_prior_predictive()- Visualization of trajectoriesQuality Assurance ✓
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.