-
Notifications
You must be signed in to change notification settings - Fork 6
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
make_table_10_gt #171
base: main
Are you sure you want to change the base?
make_table_10_gt #171
Conversation
-update argument for AESOC specification
…-create-make_table_10_gt@main
Code Coverage Summary
Diff against main
Results for commit: 09e4b51 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Test Performance Difference
Additional test case details
Results for commit 15b5d97 ♻️ This comment has been updated with latest results. |
Thank you so much for working on this @YoshitoKoujin! I have a couple of suggestions:
Is there a way to remove the numbers from these rows?
|
Thank you for your review @ayogasekaram! I've made some adjustments and removed a few rows based on your comments. Could you please take a look at the updated program? This version utilizes 'gt', but does not incorporate 'gtsummary'. However, we will need to update it once the new 'gtsummary' is released. Therefore, please consider this program as a provisional version. |
- "show_colcounts" works - remove "Risk_diff", cannot add this column - "lbl_overall" works - minor update
Unit Tests Summary 1 files 29 suites 1m 31s ⏱️ Results for commit 09e4b51. |
This PR will be revisited when the gtsummary package updates are complete and released. |
-update argument for AESOC specification
Pull Request
Closes #nnn