ARCH-commons / i2p-transform

i2b2 to PCORnet Common Data Model Transformation - requires i2b2 PCORnet ontology
Other
3 stars 16 forks source link

Numeric fields are varchar #68

Closed jklann closed 6 years ago

jklann commented 6 years ago

MEDADMIN_START_TIME MEDADMIN_STOP_TIME OBSCLIN_TIME OBSGEN_TIME

are all numeric and should be varchar

galozins commented 6 years ago

The fields are varchar(5) in PopMedNet and Pcornet CDM 4.0 specs list the columns as TEXT(5) but the EDC report seems to expect them as numeric.

jklann commented 6 years ago

The spec requires different data types in SQL and SAS, for some reason. Commit 42692cb71c858db6a3572d08e3ac7c0e545a093d fixes this in the SAS script.