Skip to content
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

Add vignettes on decorators #1353

Merged
merged 11 commits into from
Feb 25, 2025
Merged

Add vignettes on decorators #1353

merged 11 commits into from
Feb 25, 2025

Conversation

vedhav
Copy link
Contributor

@vedhav vedhav commented Feb 24, 2025

Adds vignette on how to use decorators from this package.

m7pr and others added 6 commits February 12, 2025 13:13
# Conflicts:
#	R/tm_a_gee.R
#	R/tm_g_barchart_simple.R
#	R/tm_g_ci.R
#	R/tm_g_forest_rsp.R
#	R/tm_g_forest_tte.R
#	R/tm_g_ipp.R
#	R/tm_g_km.R
#	R/tm_g_lineplot.R
#	R/tm_g_pp_adverse_events.R
#	R/tm_g_pp_patient_timeline.R
#	R/tm_g_pp_therapy.R
#	R/tm_g_pp_vitals.R
#	R/tm_t_abnormality.R
#	R/tm_t_abnormality_by_worst_grade.R
#	R/tm_t_ancova.R
#	R/tm_t_binary_outcome.R
#	R/tm_t_coxreg.R
#	R/tm_t_events.R
#	R/tm_t_events_by_grade.R
#	R/tm_t_events_patyear.R
#	R/tm_t_events_summary.R
#	R/tm_t_exposure.R
#	R/tm_t_logistic.R
#	R/tm_t_mult_events.R
#	R/tm_t_pp_basic_info.R
#	R/tm_t_pp_laboratory.R
#	R/tm_t_pp_medical_history.R
#	R/tm_t_pp_prior_medication.R
#	R/tm_t_shift_by_arm.R
#	R/tm_t_shift_by_arm_by_worst.R
#	R/tm_t_shift_by_grade.R
#	R/tm_t_smq.R
#	R/tm_t_summary.R
#	R/tm_t_summary_by.R
#	R/tm_t_tte.R
#	man/tm_a_gee.Rd
#	man/tm_g_barchart_simple.Rd
#	man/tm_g_ci.Rd
#	man/tm_g_forest_rsp.Rd
#	man/tm_g_forest_tte.Rd
#	man/tm_g_ipp.Rd
#	man/tm_g_km.Rd
#	man/tm_g_lineplot.Rd
#	man/tm_g_pp_adverse_events.Rd
#	man/tm_g_pp_patient_timeline.Rd
#	man/tm_g_pp_therapy.Rd
#	man/tm_g_pp_vitals.Rd
#	man/tm_t_abnormality.Rd
#	man/tm_t_abnormality_by_worst_grade.Rd
#	man/tm_t_ancova.Rd
#	man/tm_t_binary_outcome.Rd
#	man/tm_t_coxreg.Rd
#	man/tm_t_events.Rd
#	man/tm_t_events_by_grade.Rd
#	man/tm_t_events_patyear.Rd
#	man/tm_t_events_summary.Rd
#	man/tm_t_exposure.Rd
#	man/tm_t_logistic.Rd
#	man/tm_t_mult_events.Rd
#	man/tm_t_pp_basic_info.Rd
#	man/tm_t_pp_laboratory.Rd
#	man/tm_t_pp_medical_history.Rd
#	man/tm_t_pp_prior_medication.Rd
#	man/tm_t_shift_by_arm.Rd
#	man/tm_t_shift_by_arm_by_worst.Rd
#	man/tm_t_shift_by_grade.Rd
#	man/tm_t_smq.Rd
#	man/tm_t_summary.Rd
#	man/tm_t_summary_by.Rd
#	man/tm_t_tte.Rd
@vedhav vedhav added the core label Feb 24, 2025
Copy link
Contributor

github-actions bot commented Feb 24, 2025

Unit Tests Summary

    1 files     70 suites   1h 10m 24s ⏱️
  726 tests   726 ✅ 0 💤 0 ❌
1 986 runs  1 986 ✅ 0 💤 0 ❌

Results for commit d44ddf0.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Feb 24, 2025

Unit Test Performance Difference

Test suite performance difference
Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
examples 💔 $1.58$ $+3.96$ $0$ $-112$ $0$ $0$
shinytest2-tm_a_gee 💚 $132.40$ $-4.74$ $0$ $0$ $0$ $0$
shinytest2-tm_a_mmrm 💚 $756.73$ $-22.71$ $0$ $0$ $0$ $0$
shinytest2-tm_g_barchart_simple 💚 $232.25$ $-5.21$ $0$ $0$ $0$ $0$
shinytest2-tm_g_forest_rsp 💚 $182.74$ $-3.67$ $0$ $0$ $0$ $0$
shinytest2-tm_g_forest_tte 💚 $66.21$ $-1.69$ $0$ $0$ $0$ $0$
shinytest2-tm_g_ipp 💚 $113.51$ $-3.32$ $0$ $0$ $0$ $0$
shinytest2-tm_g_km 💚 $280.30$ $-7.86$ $0$ $0$ $0$ $0$
shinytest2-tm_g_lineplot 💚 $90.66$ $-1.89$ $0$ $0$ $0$ $0$
shinytest2-tm_g_pp_patient_timeline 💚 $250.90$ $-5.75$ $0$ $0$ $0$ $0$
shinytest2-tm_g_pp_therapy 💚 $199.48$ $-3.15$ $0$ $0$ $0$ $0$
shinytest2-tm_g_pp_vitals 💚 $89.84$ $-1.51$ $0$ $0$ $0$ $0$
shinytest2-tm_t_ancova 💚 $98.65$ $-1.52$ $0$ $0$ $0$ $0$
shinytest2-tm_t_binary_outcome 💚 $79.63$ $-1.81$ $0$ $0$ $0$ $0$
shinytest2-tm_t_coxreg 💚 $75.63$ $-1.61$ $0$ $0$ $0$ $0$
shinytest2-tm_t_events 💚 $62.89$ $-1.12$ $0$ $0$ $0$ $0$
shinytest2-tm_t_events_by_grade 💚 $92.79$ $-1.74$ $0$ $0$ $0$ $0$
shinytest2-tm_t_events_patyear 💚 $52.97$ $-1.20$ $0$ $0$ $0$ $0$
shinytest2-tm_t_events_summary 💚 $69.50$ $-1.38$ $0$ $0$ $0$ $0$
shinytest2-tm_t_exposure 💚 $82.82$ $-2.14$ $0$ $0$ $0$ $0$
shinytest2-tm_t_logistic 💚 $62.48$ $-3.23$ $0$ $0$ $0$ $0$
shinytest2-tm_t_mult_events 💚 $59.98$ $-1.33$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_basic_info 💚 $42.46$ $-1.45$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_laboratory 💚 $131.06$ $-2.91$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_medical_history 💚 $69.35$ $-2.61$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_prior_medication 💚 $80.41$ $-1.57$ $0$ $0$ $0$ $0$
shinytest2-tm_t_shift_by_arm 💚 $60.91$ $-1.02$ $0$ $0$ $0$ $0$
shinytest2-tm_t_shift_by_arm_by_worst 💚 $94.18$ $-2.17$ $0$ $0$ $0$ $0$
shinytest2-tm_t_shift_by_grade 💚 $81.77$ $-2.30$ $0$ $0$ $0$ $0$
shinytest2-tm_t_smq 💚 $61.50$ $-1.85$ $0$ $0$ $0$ $0$
shinytest2-tm_t_summary_by 💚 $82.04$ $-1.92$ $0$ $0$ $0$ $0$
shinytest2-tm_t_tte 💚 $69.95$ $-1.49$ $0$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
shinytest2-tm_a_mmrm 💚 $12.28$ $-1.14$ e2e_tm_a_mmrm_Deselection_paramcd_throws_validation_error_in_method_g_mmrm_lsmeans.
shinytest2-tm_a_mmrm 💚 $49.20$ $-1.23$ e2e_tm_a_mmrm_Validate_output_on_different_selection_on_method_g_mmrm_lsmeans.
shinytest2-tm_a_mmrm 💚 $50.41$ $-1.02$ e2e_tm_a_mmrm_Validate_output_on_different_selection_on_method_t_mmrm_lsmeans.

Results for commit a7acf90

♻️ This comment has been updated with latest results.

@vedhav vedhav enabled auto-merge (squash) February 25, 2025 05:10
@vedhav vedhav merged commit 97875bb into main Feb 25, 2025
29 checks passed
@vedhav vedhav deleted the 1475_vignettes@main branch February 25, 2025 18:50
@github-actions github-actions bot locked and limited conversation to collaborators Feb 25, 2025
@donyunardi donyunardi self-assigned this Feb 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants