insightsengineering / teal.modules.clinical

Provides teal modules for the standard clinical trials outputs
https://insightsengineering.github.io/teal.modules.clinical/
Other
32 stars 17 forks source link

Update DESCRIPTION tern to 0.9.6 #1227

Closed shajoezhu closed 2 months ago

shajoezhu commented 2 months ago

Pull Request

Fixes #nnn

github-actions[bot] commented 2 months ago

Unit Tests Summary

    1 files     70 suites   1h 4m 56s :stopwatch:   724 tests   614 :white_check_mark: 110 :zzz: 0 :x: 1 992 runs  1 767 :white_check_mark: 225 :zzz: 0 :x:

Results for commit 55c13adf.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 2 months ago

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
shinytest2-tm_g_barchart_simple 💔 $217.93$ $+3.54$ $0$ $0$ $0$ $0$
shinytest2-tm_g_ci 💔 $99.25$ $+1.61$ $0$ $0$ $0$ $0$
shinytest2-tm_g_forest_rsp 💔 $164.12$ $+4.20$ $0$ $0$ $0$ $0$
shinytest2-tm_g_km 💔 $250.30$ $+2.11$ $0$ $0$ $0$ $0$
shinytest2-tm_g_lineplot 💔 $82.24$ $+1.23$ $0$ $0$ $0$ $0$
shinytest2-tm_g_pp_patient_timeline 💔 $219.75$ $+2.51$ $0$ $0$ $0$ $0$
shinytest2-tm_g_pp_vitals 💚 $78.24$ $-1.01$ $0$ $0$ $0$ $0$
shinytest2-tm_t_abnormality 💚 $63.75$ $-1.95$ $0$ $0$ $0$ $0$
shinytest2-tm_t_events 💔 $55.10$ $+2.13$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_medical_history 💚 $62.76$ $-1.24$ $0$ $0$ $0$ $0$
shinytest2-tm_t_pp_prior_medication 💚 $75.61$ $-3.45$ $0$ $0$ $0$ $0$
shinytest2-tm_t_shift_by_arm 💚 $55.45$ $-1.93$ $0$ $0$ $0$ $0$
shinytest2-tm_t_summary 💚 $38.01$ $-1.26$ $0$ $0$ $0$ $0$

Results for commit 7ea819e4a9fa102115e93b70741aa3ded119b80f

♻️ This comment has been updated with latest results.

edelarua commented 2 months ago

Do we also want to update the rtables version to >= 0.6.10?

shajoezhu commented 2 months ago

Do we also want to update the rtables version to >= 0.6.10?

hi @edelarua , thanks for raising this. I had thought about this, but decided not to. tern 0.9.6 actually had a few fixes that were compatible with earlier rtables and formatters, so I didn't bump the rtables and formatters version in tern, so I don't think we must do here

edelarua commented 2 months ago

Ok perfect! Just wanted to double check.