You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I wrote an R script for upload the Synpuf data into CDM v5.2.2. There are some new conventions for CDMs that the ETL script doesn't yet follow. My script fixes those, but it would be nicer if they were moved into the ETL script. These are the issues I found:
Per-domain cost tables should be merged in to a single cost table. My script currently just ignores the cost tables
All DATE fields should also get a DATE_TIME field, by just copying the date, and setting time to midnight
The DRUG_EXPOSURE_END_DATE field must be populated. This can simply be done by adding the DAYS_SUPPLY to the DRUG_EXPOSURE_START_DATE.
I guess 1 is harder, but 2 and 3 are pretty trivial.
For even newer versions I think we need to start using the new TYPE_CONCEPT_ID and STATUS_CONCEPT_ID codes and fields.
The text was updated successfully, but these errors were encountered:
I wrote an R script for upload the Synpuf data into CDM v5.2.2. There are some new conventions for CDMs that the ETL script doesn't yet follow. My script fixes those, but it would be nicer if they were moved into the ETL script. These are the issues I found:
DATE
fields should also get aDATE_TIME
field, by just copying the date, and setting time to midnightDRUG_EXPOSURE_END_DATE
field must be populated. This can simply be done by adding theDAYS_SUPPLY
to theDRUG_EXPOSURE_START_DATE
.I guess 1 is harder, but 2 and 3 are pretty trivial.
For even newer versions I think we need to start using the new
TYPE_CONCEPT_ID
andSTATUS_CONCEPT_ID
codes and fields.The text was updated successfully, but these errors were encountered: