insightsengineering / scda.2022

An archive of synthetic CDISC data for 2022
https://insightsengineering.github.io/scda.2022/
Other
9 stars 3 forks source link

Initialize `rcd_2023_03_17` data #112

Closed edelarua closed 1 year ago

edelarua commented 1 year ago

Closes #109

edelarua commented 1 year ago

@Melkiades any idea why it would require lubridate for loading cached data (only when --as-cran)? https://github.com/insightsengineering/scda.2022/actions/runs/4483374122/jobs/7882588517?pr=112#step:35:108

shajoezhu commented 1 year ago

any functions that we need to import from lubridate?

Melkiades commented 1 year ago

@Melkiades any idea why it would require lubridate for loading cached data (only when --as-cran)? https://github.com/insightsengineering/scda.2022/actions/runs/4483374122/jobs/7882588517?pr=112#step:35:108

Could it be because of the metadata where there is the date and datetime class?

edelarua commented 1 year ago

@Melkiades any idea why it would require lubridate for loading cached data (only when --as-cran)? https://github.com/insightsengineering/scda.2022/actions/runs/4483374122/jobs/7882588517?pr=112#step:35:108

Could it be because of the metadata where there is the date and datetime class?

That shouldn't be the case since the date class uses as.Date and the datetime class uses as.POSIXct, both of which convert to base R classes.

EDIT: I found the issue - the study_duration_secs attribute for adsl needs to be converted to numeric.

edelarua commented 1 year ago

Unblocked by https://github.com/insightsengineering/random.cdisc.data/pull/259