Skip to content

fix: resolve transitive BLQ imputation dependencies for half.life chain (#1057)#1338

Open
wangzhengdna-lang wants to merge 4 commits into
pharmaverse:mainfrom
wangzhengdna-lang:1057-fix/blq-impute-half-life-deps
Open

fix: resolve transitive BLQ imputation dependencies for half.life chain (#1057)#1338
wangzhengdna-lang wants to merge 4 commits into
pharmaverse:mainfrom
wangzhengdna-lang:1057-fix/blq-impute-half-life-deps

refactor: use explicit exclusion set instead of max_depth in walk_for…

737a8f3
Select commit
Loading
Failed to load commit list.
Task list completed / task-list-completed succeeded Jun 4, 2026 in 0s

10 / 10 tasks completed

All tasks have been completed

Details

Required Tasks

Task Status
half.life and lambda.z retain BLQ imputation when AUC parameters are requested Completed
Purely observational parameters (cmax, tmax, tlast) still have imputation removed Completed
All existing tests pass Completed
New helper functions have dedicated unit tests Completed
Code passes lintr checks (0 lints on intervals.R) Completed
Code passes all unit tests (123 interval-related tests pass) Completed
New logic covered by unit tests — .build_rev_deps() and .walk_forward_deps() have 4 dedicated tests Completed
New logic is documented — roxygen2 @noRd docs for both helpers Completed
App or package changes are reflected in NEWS Completed
Package version is incremented Completed