Skip to content

Gt clinical tables vignette #1145

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

Merged
merged 17 commits into from
Dec 6, 2022
Merged

Conversation

alex-lauer
Copy link
Collaborator

Summary

I created a mock-up ADaM subject level dataset and vignette (article) to outline the necessary steps to create a demographic summary table and event rate analysis table with gt.

Related GitHub Issues and PRs

  • Ref: None

Checklist

alex-lauer and others added 5 commits November 14, 2022 19:37
First clinical tables vignette draft
Added a mock-up ADaM dataset and created a vignette to outline the creation of demographic summary tables and a subgroup event rate analysis table.
@CLAassistant
Copy link

CLAassistant commented Nov 24, 2022

CLA assistant check
All committers have signed the CLA.

@alex-lauer alex-lauer closed this Nov 24, 2022
@alex-lauer alex-lauer deleted the gt_clinical_tables branch November 24, 2022 19:13
@alex-lauer alex-lauer restored the gt_clinical_tables branch November 25, 2022 08:55
@alex-lauer alex-lauer reopened this Nov 25, 2022
@alex-lauer
Copy link
Collaborator Author

Hi Rich, thanks for supporting me on this project and letting me participate. {gt} is amazing - thank you so much!

@rich-iannone
Copy link
Member

Hi Rich, thanks for supporting me on this project and letting me participate. {gt} is amazing - thank you so much!

You’re welcome and I definitely appreciate the work you put into this vignette/article (many others reading it will feel the same)!

@alex-lauer alex-lauer marked this pull request as draft December 5, 2022 08:48
@alex-lauer alex-lauer marked this pull request as ready for review December 5, 2022 10:05
@alex-lauer
Copy link
Collaborator Author

Hi @rich-iannone, I finally added a description for gt_adsl in datasets.R. Thanks again for all your support!

Check for new dataset
@alex-lauer alex-lauer marked this pull request as draft December 5, 2022 10:14
@alex-lauer alex-lauer marked this pull request as ready for review December 5, 2022 15:42
@rich-iannone
Copy link
Member

rich-iannone commented Dec 5, 2022

Could we add you as an author in DESCRIPTION? If you're okay with that, put in your citation after Ellis Hughes' in this form:

    person("<first name>", "<family name>", , "<email address>", "aut",
           comment = c(ORCID = "<orcid id>"))

@rich-iannone rich-iannone self-requested a review December 5, 2022 21:57
@alex-lauer alex-lauer marked this pull request as draft December 6, 2022 10:14
@alex-lauer alex-lauer marked this pull request as ready for review December 6, 2022 12:29
Copy link
Member

@rich-iannone rich-iannone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm and thanks so much for all of this!

@rich-iannone rich-iannone merged commit d638baa into rstudio:master Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants