-
Notifications
You must be signed in to change notification settings - Fork 3
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
Closes #40 use pharmaversesdtm to host dm_peds and vs_peds #84
Changes from 6 commits
1d2557b
3c79abc
45f0aaa
4bca9ef
4eed445
935eec8
1c4d8f4
15972ab
4969902
fc90f51
a456a5b
f248704
2cd69a7
5623fe6
716143e
37da7e7
fcffc5d
6f53497
01db875
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -116,10 +116,11 @@ | |
#' library(lubridate, warn.conflicts = FALSE) | ||
#' library(rlang, warn.conflicts = FALSE) | ||
#' library(admiral, warn.conflicts = FALSE) | ||
#' library(pharmaversesdtm, warn.conflicts = FALSE) | ||
#' | ||
#' advs <- dm_peds %>% | ||
#' advs <- pharmaversesdtm::dm_peds %>% | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should be able to remove There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. to demonstrate it works being in pharmaversesdtm if its actually in there, yes |
||
#' select(USUBJID, BRTHDTC, SEX) %>% | ||
#' right_join(., vs_peds, by = "USUBJID") %>% | ||
#' right_join(., pharmaversesdtm::vs_peds, by = "USUBJID") %>% | ||
#' mutate( | ||
#' VSDT = ymd(VSDTC), | ||
#' BRTHDT = ymd(BRTHDTC) | ||
|
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zdz2101 this bit can be removed and you could get this and pharmaverse/pharmaversesdtm#115 ready for review now, and think should be good timing for me and @Fanny-Gautier to take a look in the coming weeks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the steps go:
pharmaversdtm::
as well, because I think it'll still rely on a remotes until the pharmaversesdtm package is re-released to a new versionThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plus Step 4) add an
adsl_peds
andadvs_peds
to pharmaverseadam. we can make an issue for this later after all these stepsThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As for
adsl_peds
andadvs_peds
it seems there was some ongoing discussion about what is appropriate to house in there as the datasets for that package are made from the templates inadmiral
see discussion from this issueThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think they mean any admiral package templates incl extensions here, as they do have ADaMs created from ophtha, onco, vaccine templates for example at https://github.com/pharmaverse/pharmaverseadam/tree/main/data