-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature Request: Add/Update SDTM test data for {admiralpeds} #88
Comments
@rossfarrugia we would like to release pharmaversesdtm 1.0 at the start of June. Will the test data be ready by then? Otherwise would need to add in a later release and temporarily keep in admiralpeds. |
@manciniedoardo - @kabis-ops has created this at https://github.com/pharmaverse/admiralpeds/tree/main/data - see |
Thanks @rossfarrugia - I will label this issue as Post 1.0. Perhaps it would be good then to also make a companion issue (and link to this one) on the admiralpeds side for when you do have a chance to move the data over. |
yes we have it here pharmaverse/admiralpeds#36, will let you know if anything changes. We might be able to squeeze it in earlier as its impacting our template/vignette anyway as we don't have an |
I think this one can stay open for now and we'll move these datasets over after we do admiralpeds 0.1.0 release in July. It corresponds now to this issue pharmaverse/admiralpeds#40 |
Thanks for re-opening. It was linked to the wrong PR and got closed when that PR was merged, all good now. |
Closes #88 Use pharmaversesdtm to host admiralpeds test data
Feature Idea
Add/Update SDTM test data for {admiralpeds}
Relevant Input
N/A
Relevant Output
N/A
Reproducible Example/Pseudo Code
N/A
The text was updated successfully, but these errors were encountered: