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

[Feature Request] Missing variables in the ADCM dataset #77

Closed
1 of 3 tasks
edelarua opened this issue Jul 26, 2024 · 7 comments
Closed
1 of 3 tasks

[Feature Request] Missing variables in the ADCM dataset #77

edelarua opened this issue Jul 26, 2024 · 7 comments
Assignees

Comments

@edelarua
Copy link

edelarua commented Jul 26, 2024

Hello,

While testing safety templates in scda.test, we found that the following variables are missing from the ADCM dataset:

If you require any further details please let me/the SME team know.

Thanks!!

@kaz462
Copy link
Contributor

kaz462 commented Aug 7, 2024

Hi @edelarua @manciniedoardo,

  • could you help suggest how to derive ATC1/2/3/4 based on the following variables from current cm?
 [1] "STUDYID"  "DOMAIN"   "USUBJID"  "CMSEQ"    "CMSPID"   "CMTRT"    "CMDECOD"  "CMINDC"   "CMCLAS"  
[10] "CMDOSE"   "CMDOSU"   "CMDOSFRQ" "CMROUTE"  "VISITNUM" "VISIT"    "VISITDY"  "CMDTC"    "CMSTDTC" 
[19] "CMENDTC"  "CMSTDY"   "CMENDY" 
  • Is "Adverse Event Treatment Relationship" the label for ATIREL?

Thanks!

@manciniedoardo
Copy link
Collaborator

@kaz462 I think the label for ATIREL should be "Time Relative to Reference"

@manciniedoardo
Copy link
Collaborator

For the first bullet point, let's bring up at today's admiral call.

@edelarua
Copy link
Author

edelarua commented Aug 7, 2024

Hi @kaz462,

  • could you help suggest how to derive ATC1/2/3/4 based on the following variables from current cm?
 [1] "STUDYID"  "DOMAIN"   "USUBJID"  "CMSEQ"    "CMSPID"   "CMTRT"    "CMDECOD"  "CMINDC"   "CMCLAS"  
[10] "CMDOSE"   "CMDOSU"   "CMDOSFRQ" "CMROUTE"  "VISITNUM" "VISIT"    "VISITDY"  "CMDTC"    "CMSTDTC" 
[19] "CMENDTC"  "CMSTDY"   "CMENDY" 

The ATC1/2/3/4 variables directly correlate with the CMDECOD/CMCLAS variables. For example, in random.cdisc.data we have the following synthetic data:

Screenshot 2024-08-07 150043

  • Is "Adverse Event Treatment Relationship" the label for ATIREL?

ATIREL in our adcm data has the label "Time Relation of Medication" (though I think "Time Relative to Reference" is the more general, data non-specific variable name) with two levels - CONCOMITANT and PRIOR.

@kaz462
Copy link
Contributor

kaz462 commented Aug 7, 2024

@edelarua Thank you for clarification! I was thinking to get the real ATC code, but it seems you are okay with synthetic/fake data as long as the values populated correlate with CMDECOD/CMCAT?

@manciniedoardo maybe we can simply simulate the ATC variables instead of merging with whodrug/facm from derive_vars_atc to avoid the complication, what do you think?
copy @pharmaverse/admiral @pharmaverse/admiral_comm

@edelarua
Copy link
Author

edelarua commented Aug 7, 2024

@edelarua Thank you for clarification! I was thinking to get the real ATC code, but it seems you are okay with synthetic/fake data as long as the values populated correlate with CMDECOD/CMCAT?

For our purposes, that should be fine.

Thanks!

@bms63
Copy link
Collaborator

bms63 commented Aug 7, 2024

@edelarua Thank you for clarification! I was thinking to get the real ATC code, but it seems you are okay with synthetic/fake data as long as the values populated correlate with CMDECOD/CMCAT?

@manciniedoardo maybe we can simply simulate the ATC variables instead of merging with whodrug/facm from derive_vars_atc to avoid the complication, what do you think? copy @pharmaverse/admiral @pharmaverse/admiral_comm

I'm okay with simulated data if it makes sense! Can something simple be put together and we can discuss a little more?

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

No branches or pull requests

4 participants