Skip to content
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

ADVS template for growth parameters #32

Open
rossfarrugia opened this issue Apr 30, 2024 · 10 comments · May be fixed by #38
Open

ADVS template for growth parameters #32

rossfarrugia opened this issue Apr 30, 2024 · 10 comments · May be fixed by #38
Assignees

Comments

@rossfarrugia
Copy link
Collaborator

rossfarrugia commented Apr 30, 2024

See the outline document in MS Teams area. Steps 5 and 6 rely on the new functions being available so placeholders will be needed for now.

The main part will be Step 4 for preparing the metadata into a consistent structure covering the below points:

PROPOSAL FOR BY AGE: In this step for each of BMI/HEIGHT/WEIGHT, we would combine WHO <2yrs and CDC >=2 data into one single consistently structured meta dataset (with SEX as a variable - with M and F values, AGE in days via a conversion factor). Clearly comment to make clear here our default reference source, in our case WHO <2 yrs and CDC >=2, and the fact that any user could replace this with their own chosen metadata. We can explain this then more in the vignette.

PROPOSAL FOR THE WHO-ONLY DATA: In this step for each of HEAD CIRCUMFERENCE and WEIGHT (BY LENGTH/HEIGHT), we would combine the boys and girls data into one single consistently structured meta dataset (with SEX as a variable - with M and F values). For WEIGHT (BY LENGTH/HEIGHT) additionally then combine the LENGTH and HEIGHT data into one single dataset with additional variable MEASURE to denote LENGTH or HEIGHT and HEIGHT_LENGTH variable showing the length/height values.

The naming of these meta datasets should still clearly convey which parameter they each relate to, so then no need to add PARAMCD here as a variable in these datasets.

@Fanny-Gautier
Copy link

Fanny-Gautier commented May 2, 2024

From the above we understood to create in total 5 metadata sets from the 15 datasets as following:

  1. By age, containing:
    a) bmi_by_age: BMI related data (WHO_bmi_for_age_boys.rda, WHO_bmi_for_age_girls.rda, cdc_bmiage.rda) + variable SEX (converting 1 to "M"/ 2 to "F" for cdc data, and adding "M" for who_boys data / "F" for who_girls data) + variable AGE (in days)
    b) height_by_age: HEIGHT related data (who_lgth_ht_for_age_boys.rda, who_lght_ht_for_age_girls.rda, cdc_htage.rda) + variable SEX (converting 1 to "M"/ 2 to "F" for cdc data, and adding "M" for who_boys data / "F" for who_girls data) + variable AGE (in days)
    c) weight_by_age: WEIGHT related data (who_wt_for_age_boys.rda, who_wt_for_age_girls.rda, cdc_wtage.rda) + variable SEX (converting 1 to "M"/ 2 to "F" for cdc data, and adding "M" for who_boys data / "F" for who_girls data) + variable AGE (in days)

  2. WHO: HC and WT only:
    a) who_hc: HEAD CIRCUMFERENCE related data (who_hc_for_age_boys.rda, who_hc_for_age_girls.rda) which include the variable AGE in days + variable SEX (adding "M" for who_boys data / "F" for who_girls data)
    b) _who_wt_lgt: WEIGHT/HEIGHT related data (who_wt_for_ht_boys.rda, who_wt_for_ht_girls.rda, who_wt_for_lgth_boys.rda, who_wt_for_lgth_girls.rda) + variable MEASURE set to "HEIGHT" for wt_for_ht data, and set to "LENGTH" for wt_for_lgth data + variable SEX ("M" for boys data / "F" for girls data) + variables "Height" and "Length" renamed as the single one "HEIGHT_LENGTH"

Could you please confirm our understanding ?

Thank you.

@rossfarrugia
Copy link
Collaborator Author

rossfarrugia commented May 2, 2024

@Fanny-Gautier yes, that's right, and some additional points:

  • the by age files only need to keep the WHO data up to <2 years (i.e. 729 days) - as from 2 onwards we'd use the CDC data
  • minor comment but maybe use the naming _for_ instead of _by_ as that's how Pierre set up the WHO metafiles in our package.

This should be done in a file named inst/templates/ad_advs.R with the earlier generic ADVS steps copied from https://github.com/pharmaverse/admiral/blob/main/inst/templates/ad_advs.R as per the outline doc on MS teams.

@Fanny-Gautier
Copy link

Fanny-Gautier commented May 6, 2024

is 2 years <731 days or <= 729 ? In the ADVS specs we talk about <731 days...
30.4375 x 24 = 730.5 , knowing that it is strictly under 2 years old, I would have keep till 730 days included. Please confirm.

@Fanny-Gautier
Copy link

Converted months in days gives 730.5. So for WHO data we would have till 729 days included:
image
and for CDC data it's from 730.5 days:
image

Is it correct this way? in that case could you please clarify the ADVS specs where <731 days is mentioned ? thanks.

@rossfarrugia
Copy link
Collaborator Author

is 2 years <731 days or <= 729 ? In the ADVS specs we talk about <731 days... 30.4375 x 24 = 730.5 , knowing that it is strictly under 2 years old, I would have keep till 730 days included. Please confirm.

@Fanny-Gautier given to account for leap year we usually assume a year is 365.25 days so that means we should include up to 730 days from WHO and 731 days onwards from CDC. sorry for confusing above with the mention of 729

@Fanny-Gautier
Copy link

ADVS template: could you please guide us for the merge of ADSL variables (TRTSDT) since these are not the same patients as in dm_peds ? I am not sure to understand how to proceed with step 3 (Computing AAGECUR) and merge ADSL variables from step 1b.

@rossfarrugia
Copy link
Collaborator Author

@Fanny-Gautier i spotted same today (see discussion at pharmaverse/pharmaversesdtm#88). for now just use the dm_peds directly (and ignore mention of ADSL). we might need to come back to this later and update the test data and run a new ADSL from it to use, as we usually try to extend the same CDISC test pilot study data as in {pharmaversesdtm}.

@Fanny-Gautier
Copy link

To compute AAGECUR, do we impute the partial Birth Dates ? If so, which approach is more appropriate for pediatric patients ? is birth date not before a list of minimum dates ok ? Also for partial Analysis date if any, do we impute to first day of the month ?

Regarding Birth date imputation, I am not sure why BRTHDT is not found while well merged as first step to VS:
image

@rossfarrugia
Copy link
Collaborator Author

rossfarrugia commented May 7, 2024

I wouldn't impute partial birth dates for the template - let's keep this example simple. I saw the test data has one such case but think we could update this later to make all dates complete. I'd expect most trials have complete birth dates and I wouldn't want to estimate what the appropriate imputation rule might be if they didn't.

For the issue with the code, feel free to email me the full code you're running and I could try and figure out on my side. I don't see anything obvious from your screenshot.

Lina2689 added a commit that referenced this issue May 8, 2024
Fanny-Gautier added a commit that referenced this issue May 8, 2024
Fanny-Gautier added a commit that referenced this issue May 8, 2024
Lina2689 added a commit that referenced this issue May 10, 2024
… Derived z-score and percentile parameters for BMI, WEIGHT and Head circumference.
Fanny-Gautier added a commit that referenced this issue May 10, 2024
@Fanny-Gautier Fanny-Gautier linked a pull request May 10, 2024 that will close this issue
14 tasks
Fanny-Gautier added a commit that referenced this issue May 10, 2024
Fanny-Gautier added a commit that referenced this issue May 13, 2024
Lina2689 added a commit that referenced this issue May 15, 2024
…GHT variable. and modified the function call derive_params_growth_age to keep the derived parameters and corresponding input records only.
@Fanny-Gautier
Copy link

Delete line of code mutate(AVALC = VSSTRESC) as per Nancy's comment

Fanny-Gautier added a commit that referenced this issue Jun 6, 2024
Co-authored-by: Ross Farrugia <82581364+rossfarrugia@users.noreply.github.com>
Fanny-Gautier added a commit that referenced this issue Jun 6, 2024
Fanny-Gautier added a commit that referenced this issue Jun 10, 2024
…d HEIGHT_LENGTHU in both who_wt_for_lgth and who_wt_for_ht metadata
Lina2689 added a commit that referenced this issue Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

3 participants